<div>Hi,</div><div><br></div><div>I've been using Live555 and I like it. Despite things are starting to run smooth, I have a suggestion to improve the logging </div><div>in the library, this will be usefull when tracking difficult problems that happen rarely.</div>
<div><br></div><div>Using the current codebase, I implemented logging using a BasicUsageEnvironment derived class, </div><div>and so I received some logging from the library classes as well. However, live555 is also sending usefull info to stderr, </div>
<div>and to catch that as well, one also has to redirect stderr. These statements do not contain time info, and are only triggered in </div><div>debug builds.</div><div><br></div><div>For example in RTSPServer.cpp :</div>
<div><br></div><div>#ifdef DEBUG</div><div>  fprintf(stderr, "parseRTSPRequestString() failed\n");</div><div>#endif</div><div><br></div><div><br></div><div>#ifdef DEBUG</div><div>  envir() << "accept()ed connection from " << our_inet_ntoa(clientAddr.sin_addr) << "\n";</div>
<div>#endif</div><div><br></div><div>So my suggestion is to :</div><div><br></div><div>- Do all logging in the same way (through the environment class)</div><div>- Always add timestamps</div><div>- Add levels of logging (verbose, info, error )</div>
<div>- Always log, not only in debug builds, the specific environment class can decide what to do with it</div><div>- For conveniance, provide a basiclogger that does the writing to the logfile(s)</div><div><br></div><div>
Regards,</div><div>Steph</div>