Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-06 08:38:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Remove last_access_time from TableEntryVal.

Browse source 
Turns out - this was no longer used. And it takes up at least 8 bytes
of space for every single table/set entry.
This commit is contained in:
Johanna Amann 2020-06-18 22:59:08 +00:00
parent 7ae5589469
commit 64af3cdf05
2 changed files with 0 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/Val.cc
View file

@ -1307,7 +1307,6 @@ unsigned int ListVal::MemoryAllocation() const
TableEntryVal* TableEntryVal::Clone(Val::CloneState* state) TableEntryVal* TableEntryVal::Clone(Val::CloneState* state)
{ {
auto rval = new TableEntryVal(val ? val->Clone(state) : nullptr); auto rval = new TableEntryVal(val ? val->Clone(state) : nullptr);
rval->last_access_time = last_access_time;
rval->expire_access_time = expire_access_time; rval->expire_access_time = expire_access_time;
return rval; return rval;
} }

3
src/Val.h
View file

@ -722,7 +722,6 @@ public:
explicit TableEntryVal(IntrusivePtr<Val> v) explicit TableEntryVal(IntrusivePtr<Val> v)
: val(std::move(v)) : val(std::move(v))
{ {
last_access_time = network_time;
expire_access_time = expire_access_time =
int(network_time - bro_start_network_time); int(network_time - bro_start_network_time);
} }
@ -745,8 +744,6 @@ protected:
friend class TableVal; friend class TableVal;
IntrusivePtr<Val> val; IntrusivePtr<Val> val;
double last_access_time;
// The next entry stores seconds since Bro's start. We use ints here // The next entry stores seconds since Bro's start. We use ints here
// to save a few bytes, as we do not need a high resolution for these // to save a few bytes, as we do not need a high resolution for these
// anyway. // anyway.