From 64af3cdf05bc40c88f2c2488c5eef60764b43c86 Mon Sep 17 00:00:00 2001
From: Johanna Amann <johanna@corelight.com>
Date: Thu, 18 Jun 2020 22:59:08 +0000
Subject: [PATCH] Remove last_access_time from TableEntryVal.

Turns out - this was no longer used. And it takes up at least 8 bytes
of space for every single table/set entry.
---
 src/Val.cc | 1 -
 src/Val.h  | 3 ---
 2 files changed, 4 deletions(-)

diff --git a/src/Val.cc b/src/Val.cc
index 0823c56a70..7d0fbe610f 100644
--- a/src/Val.cc
+++ b/src/Val.cc
@@ -1307,7 +1307,6 @@ unsigned int ListVal::MemoryAllocation() const
 TableEntryVal* TableEntryVal::Clone(Val::CloneState* state)
 	{
 	auto rval = new TableEntryVal(val ? val->Clone(state) : nullptr);
-	rval->last_access_time = last_access_time;
 	rval->expire_access_time = expire_access_time;
 	return rval;
 	}
diff --git a/src/Val.h b/src/Val.h
index 84afb01e51..d76d36d25a 100644
--- a/src/Val.h
+++ b/src/Val.h
@@ -722,7 +722,6 @@ public:
 	explicit TableEntryVal(IntrusivePtr<Val> v)
 		: val(std::move(v))
 		{
-		last_access_time = network_time;
 		expire_access_time =
 			int(network_time - bro_start_network_time);
 		}
@@ -745,8 +744,6 @@ protected:
 	friend class TableVal;
 
 	IntrusivePtr<Val> val;
-	double last_access_time;
-
 	// 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
 	// anyway.