From bd3bcb71f4f526fdbc04ab37c830ff6da0504604 Mon Sep 17 00:00:00 2001 From: Richard Barnes Date: Tue, 12 Dec 2023 13:19:55 -0800 Subject: [PATCH] Remove extra semi colon from oil/fbpkg_fuse/fs_item.cpp Summary: `-Wextra-semi` or `-Wextra-semi-stmt` If the code compiles, this is safe to land. Reviewed By: palmje Differential Revision: D51995036 fbshipit-source-id: e66f8b9775b48e55c24118d5d7d455b4cf3eaca5 --- src/oomd/engine/BasePlugin.h | 2 +- src/oomd/util/ScopeGuard.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/oomd/engine/BasePlugin.h b/src/oomd/engine/BasePlugin.h index 866921f6..dd45e63e 100644 --- a/src/oomd/engine/BasePlugin.h +++ b/src/oomd/engine/BasePlugin.h @@ -58,7 +58,7 @@ class BasePlugin { * because it is guaranteed to be called each interval, but run() may not. * Therefore, this function should be lightweight, i.e. no sleep() inside. */ - virtual void prerun(OomdContext& context){}; + virtual void prerun(OomdContext& context) {} /* * This is the main work method every plugin will implement. diff --git a/src/oomd/util/ScopeGuard.h b/src/oomd/util/ScopeGuard.h index ccc33dba..ecbecc71 100644 --- a/src/oomd/util/ScopeGuard.h +++ b/src/oomd/util/ScopeGuard.h @@ -32,13 +32,13 @@ class ScopeGuard { public: explicit ScopeGuard(std::function fn) { fn_ = fn; - }; + } ~ScopeGuard() { if (fn_) { fn_(); } - }; + } private: std::function fn_;