Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-26 01:51:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

[wpiutil] Logger: Conditionalize around WPI_LOG (#4431)

Browse Source 
The FMT_STRING() call used in the macro does formatter initialization,
overhead that's not required if Log() is not going to be called.
This commit is contained in:
Peter Johnson
2022-09-22 23:14:46 -07:00
committed by GitHub
parent 8767e4a941
commit 5ac658c8f0
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
wpiutil/src/main/native/include/wpi/Logger.h
View File

@@ -60,8 +60,11 @@ class Logger {
unsigned int m_min_level = 20;
};
#define WPI_LOG(logger_inst, level, format, ...) \
(logger_inst).Log(level, __FILE__, __LINE__, FMT_STRING(format), __VA_ARGS__)
#define WPI_LOG(logger_inst, level, format, ...) \
if ((logger_inst).HasLogger() && level >= (logger_inst).min_level()) { \
(logger_inst) \
.Log(level, __FILE__, __LINE__, FMT_STRING(format), __VA_ARGS__); \
}
#define WPI_ERROR(inst, format, ...) \
WPI_LOG(inst, ::wpi::WPI_LOG_ERROR, format, __VA_ARGS__)