From 1258c7aabee27bed0fc48bab80a374d004012c36 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Fri, 31 Aug 2012 14:21:16 +0200 Subject: [PATCH 1/1] Don't need to memset HashTable Default c'tor Entry() already initializes to zero all its POD members. No functional change. Signed-off-by: Marco Costalba --- src/misc.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/misc.h b/src/misc.h index 24d41d1a..424a73b2 100644 --- a/src/misc.h +++ b/src/misc.h @@ -47,7 +47,7 @@ struct Log : public std::ofstream { struct Time { void restart() { system_time(&t); } int64_t msec() const { return time_to_msec(t); } - int elapsed() const { return int(current_time().msec() - time_to_msec(t)); } + int elapsed() const { return int(current_time().msec() - msec()); } static Time current_time() { Time t; t.restart(); return t; } @@ -58,7 +58,7 @@ private: template struct HashTable { - HashTable() : e(Size, Entry()) { memset(&e[0], 0, sizeof(Entry) * Size); } + HashTable() : e(Size, Entry()) {} Entry* operator[](Key k) { return &e[(uint32_t)k & (Size - 1)]; } private: -- 2.39.2