[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH] strftime()



On Wed, Aug 12, 2009 at 02:05:58PM +0200, Andreas Freimuth wrote:
> Du verwendst glaub ich irgendwie itoa falsch.
> Laut Standard ist das: char* itoa(value,dest,base);
> du übergibst ihm Wilde dest-pointer und als Base, die Anzahl der Zeichen
> die er daraus machen soll.

itoa ist broken by design, weil es keine Stringlänge nimmt. Die bessere
Alternative ist snprintf.

> 
> Außerdem hast du manchmal <tabs> drin und whitespace am zeilenende
> (besonders in leerzeilen). Das mochte der taljeth bei meinem patch gar
> nicht.

Genau. ;-)

Falls einer von euch vim benutzt, schaut euch mal im Wiki die Seite
http://lowlevel.brainsware.org/wiki/index.php/Benutzer:FreakyPenguin
an, da gibt es eine schöne Konfigurationsdatei, mit der
Whitespace-Fehler sofort mit roter Hintergrundfarbe markiert werden.