[Live-devel] [PATCH] Enable logging levels, part1

David Stegbauer david.stegbauer at acision.com
Fri Jul 2 02:13:50 PDT 2010


Hi Ross,

Thank you for long reply. I understand how UsageEnvironment works and I also 
know how to derive and re implement "operator<<" virtual functions.

> Having said all this, I do admit, though, that the
> diagnostic/debugging output in the provided code is a bit
> inconsistent right now

I'd rather say insufficient.
For example in RTSPClient.cpp there are
if (fVerbosityLevel >= 1) envir() << something;
all over the code.
But IMHO it looks not enough fine grained. Some of such "messages" are at debug 
level, some are at info level and some could even be on warning level.

Even more important is, that my code (like MyUsageEnvironment)  does not 
receive "verbosity level" in my operator<< for particular messages . So I'm 
not able to fiter them (or redirect to different destinations - console, file, 
syslog, ...).

I think proper logging is vital part of any non-trivial code. I'm willing to 
cooperate with you to introduce it in live555 library.

> But I'd prefer not to make any such changes right now

Well, but proper logging is really important for me right now. I have feeling 
you are actively trying to discourage me from making any changes to the 
library.

So the crucial questions are:
Are you willing to discuss modifications?
Are you willing to incorporate patches?

Best Regards
David Stegbauer

This e-mail and any attachment is for authorised use by the intended recipient(s) only. It may contain proprietary material, confidential information and/or be subject to legal privilege. It should not be copied, disclosed to, retained or used by, any other party. If you are not an intended recipient then please promptly delete this e-mail and any attachment and all copies and inform the sender. Thank you.




More information about the live-devel mailing list