diff options
| author | Mike Leach <mike.leach@linaro.org> | 2021-09-20 14:52:32 +0100 |
|---|---|---|
| committer | Mike Leach <mike.leach@linaro.org> | 2021-10-10 10:04:22 +0100 |
| commit | 1be557e437a08446177b78538f99dd2a3b40c0a7 (patch) | |
| tree | 0334e4d079ad22f7e500f3af18a973bc91be008b | |
| parent | ccc11fec6315dc29a1d6786c2bccd7aabeb8f3c2 (diff) | |
build: tests: Fix build warnings in snapshot parser.
Moved static vars into .cpp file from .h to remove warnings when
including .h file in client code.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
| -rw-r--r-- | decoder/tests/snapshot_parser_lib/include/snapshot_parser.h | 6 | ||||
| -rw-r--r-- | decoder/tests/snapshot_parser_lib/source/snapshot_parser.cpp | 4 |
2 files changed, 7 insertions, 3 deletions
diff --git a/decoder/tests/snapshot_parser_lib/include/snapshot_parser.h b/decoder/tests/snapshot_parser_lib/include/snapshot_parser.h index 8b9171255d0c..9e5b37189099 100644 --- a/decoder/tests/snapshot_parser_lib/include/snapshot_parser.h +++ b/decoder/tests/snapshot_parser_lib/include/snapshot_parser.h @@ -135,9 +135,9 @@ namespace Parser std::vector<std::string> GetBufferNameList(ParsedTrace &metadata); - static ITraceErrorLog *s_pErrorLogger = 0; - static ocsd_hndl_err_log_t s_errlog_handle = 0; - static bool s_verbose_logging = true; + //static ITraceErrorLog *s_pErrorLogger = 0; + //static ocsd_hndl_err_log_t s_errlog_handle = 0; + //static bool s_verbose_logging = true; void SetIErrorLogger(ITraceErrorLog *i_err_log); void SetVerboseLogging(bool verbose); diff --git a/decoder/tests/snapshot_parser_lib/source/snapshot_parser.cpp b/decoder/tests/snapshot_parser_lib/source/snapshot_parser.cpp index 6e62d1e200c2..4570700dd4c8 100644 --- a/decoder/tests/snapshot_parser_lib/source/snapshot_parser.cpp +++ b/decoder/tests/snapshot_parser_lib/source/snapshot_parser.cpp @@ -49,6 +49,10 @@ using namespace Parser; #include "opencsd.h" +static ITraceErrorLog *s_pErrorLogger = 0; +static ocsd_hndl_err_log_t s_errlog_handle = 0; +static bool s_verbose_logging = true; + /************************************************************************* * Note, this file handles the parsring of the general (device specific) * ini file and the (much smaller) device_list file |