https://sourceforge.net/p/bartlby/bugs/Recent changes to bugsenMon, 17 Mar 2008 09:01:11 -0000https://sourceforge.net/p/bartlby/bugs/2/<div class="markdown_content"><p>I have seen some fixed integers in a source file.<br /> <a href="http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/config.c?revision=1.12&amp;view=markup">http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/config.c?revision=1.12&amp;view=markup</a></p> <p>They are unnecessarily hard to understand.<br /> <a href="http://en.wikipedia.org/wiki/Magic_number_%28programming%29#Unnamed_numerical_constant" rel="nofollow">http://en.wikipedia.org/wiki/Magic_number_%28programming%29#Unnamed_numerical_constant</a></p> <p>I suggest to improve the source code by the use of the sizeof() operator.</p></div>Markus ElfringMon, 17 Mar 2008 09:01:11 -0000https://sourceforge.net4833935ddb586e5e871c04d353bd71574f338b11https://sourceforge.net/p/bartlby/bugs/1/<div class="markdown_content"><p>Some checks for return codes are missing.</p> <p>Examples:<br /> Would you like to add more error handling for return values from "malloc" like in the function "bartlby_check_sirene" and from "fwrite" in the function "bartlby_pre_init"?<br /> <a href="http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/ack.c?revision=1.2&amp;view=markup">http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/ack.c?revision=1.2&amp;view=markup</a><br /> <a href="http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/daemon.c?revision=1.11&amp;view=markup">http://bartlby.cvs.sourceforge.net/bartlby/bartlby-core/src/daemon.c?revision=1.11&amp;view=markup</a></p></div>Markus ElfringTue, 27 Feb 2007 14:46:11 -0000https://sourceforge.netf1d88155c21b2c3560ee24df9b72305b8d99b709