[Tux4kids-tuxtype-dev] [Patch] Replace printf() with fprintf()

Jerome Portal jerome.portal.2 at gmail.com
Mon Apr 15 13:26:09 UTC 2013


Thanks for your answer ! Here is my freshly created github account, with
the commit list on tuxtype fork :
https://github.com/OjeromeO/tuxtype/commits/jerome-dev


2013/4/15 Brendan Luchen <cheezmeister at gmail.com>

> Jerome,
>
>
>  There were still some printf(...) in tuxtype code, so I replaced them
>> using the fprintf(stderr, ...) that are used elsewhere. Here is the diff :
>>
>>  Thanks! Since you made a few patches, any chance of forking the repo to
> github or similar, and committing your changes so they can be pulled all
> together? Hand patching files is messy business (which is another way of
> saying "I'm lazy")...
>
>
>> I also saw that some parts of the code are using a #ifdef DEBUG { ... }
>> structure to log messages. Wouldn't it be better to replace them with
>> DEBUGCODE { ... } (or LOG { ... }, if possible) ?
>>
>>  Probably :) There's plenty of cruft hanging around, but it's mostly
> harmless (TM).
>
> -Brendan
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/tux4kids-tuxtype-dev/attachments/20130415/605d3415/attachment.html>


More information about the Tux4kids-tuxtype-dev mailing list