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:
pgpA3duqSeblE.pgp
Description: PGP signature