Hello,
I was looking over the Logger class and was wondering if anyone had any
objections to moving the declaration of some of Logger's internal
classes (SyncSink, SeverityFilter, LogFilter) from Logger.h into
Logger.cc. I'd also like to move the visibility of some of the static
fields of Logger (syncSink, syncBuffer, and logStream) into Logger.cc.
All of these seem like pieces that the Logger needs in order to operate,
but that clients of Logger really shouldn't be messing with.
In particular, I'd like to know if Bill (author) or Serge (reviewer) can
point out what I missed. Perhaps there is flexibility to be had in
exposing those classes/fields in Logger's public interface?
Thanks!
-Daniel, on a mission to clean up Qserv code.
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the QSERV-L list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1
|