From 8ac9b32779001400d760d868df6cd49af343035a Mon Sep 17 00:00:00 2001 From: Ramanpreet Nara Date: Tue, 19 Nov 2019 10:48:51 -0800 Subject: [PATCH] Remove __unused annotation from ConcreteSystraceSection ctor args Summary: The constructor arguments to `ConcreteSystraceSection` are actually used. It seems like they were accidentally marked unused in D14181748. Changelog: [iOS][Fixed] - Remove __unused annotation from ConcreteSystraceSection ctor args. Reviewed By: PeteTheHeat Differential Revision: D18574190 fbshipit-source-id: 38d58da794341d4ecc52f3bc16e05ef2757cca1d --- ReactCommon/cxxreact/SystraceSection.h | 27 +++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/ReactCommon/cxxreact/SystraceSection.h b/ReactCommon/cxxreact/SystraceSection.h index 90a0cd30a31..97f93af0b60 100644 --- a/ReactCommon/cxxreact/SystraceSection.h +++ b/ReactCommon/cxxreact/SystraceSection.h @@ -25,26 +25,27 @@ namespace react { */ #ifdef WITH_FBSYSTRACE struct ConcreteSystraceSection { -public: - template - explicit - ConcreteSystraceSection(__unused const char* name, __unused ConvertsToStringPiece&&... args) - : m_section(TRACE_TAG_REACT_CXX_BRIDGE, name, args...) - {} + public: + template + explicit ConcreteSystraceSection( + const char *name, + ConvertsToStringPiece &&... args) + : m_section(TRACE_TAG_REACT_CXX_BRIDGE, name, args...) {} -private: + private: fbsystrace::FbSystraceSection m_section; }; using SystraceSection = ConcreteSystraceSection; #else struct DummySystraceSection { -public: - template - explicit - DummySystraceSection(__unused const char* name, __unused ConvertsToStringPiece&&... args) - {} + public: + template + explicit DummySystraceSection( + __unused const char *name, + __unused ConvertsToStringPiece &&... args) {} }; using SystraceSection = DummySystraceSection; #endif -}} +} // namespace react +} // namespace facebook