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

Re: [tyndur-devel] [PATCH 1/3] Pascal-RTL: Unterstuetzung fuer mehrsprachige Programme



On Sun, Sep 27 08:22, Kevin Wolf wrote:
> On Sun, Sep 27, 2009 at 01:30:47AM +0200, Antoine Kaufmann wrote:
> > On Sat, Sep 26 16:35, Kevin Wolf wrote:
> > > + Pascal-RTL: Unit multilang
> > > 
> > > Signed-off-by: Kevin Wolf <kevin@xxxxxxxxxx>
> > > ....
> > > +                // Nummer des Arguments bestimmen
> > > +                colon := Pos(':', PChar(@fmt[i + 2]));
> > > +                num := StrToInt(Copy(fmt, i + 2, colon - 1));
> > 
> > Hm ich glaube wenn da der Formatstring kaputt ist dürfte das rot geben,
> > oder? Also ohne Doppelpunkt. Ich glaube an Fehlerbehandlung fehlt da
> > weiter unten auch noch ein Bisschen was.
> 
> Wir sind hier doch nicht in C ;-)
> 
> Pos gibt 0 zurück, weil kein Doppelpunkt, damit baut Copy einen leeren
> String und StrToInt wirft einen EConvertError. Was eigentlich gar nicht
> so doof ist.
> 
> Gefährlich wird es eigentlich immer nur, wenn da was von PChar steht.
> Aber die Stellen müssten eigentlich alle gut sein. Läuft halt ein
> bisschen auf Garbage In, Garbage Out raus...

Ich kann doch kein Pascal. ;-) Aber dann halt, wenns denn sein muss:
Acked-by: Antoine Kaufmann <toni@xxxxxxxxxx>

-- 
Antoine Kaufmann
<toni@xxxxxxxxxxxxxxxx>

Attachment: pgpflAc8e2TGF.pgp
Description: PGP signature