Op zondag 22-03-2009 om 18:40 uur [tijdzone +0800], schreef Funky Beast: > Joost van der Sluis wrote: > > Op zondag 22-03-2009 om 17:37 uur [tijdzone +0800], schreef Funky Beast: > >> I found the culprit. > >> > >> In TIBConnection.PrepareStatement's variable section, x was declared > >> as a shortint > >> which would cause problem when its used for allocating memmory for the > >> SQLVAR elements, > >> when there are more than 128 fields. > >> > >> Also same as above in TIBConnection.FreeSQLDABuffer. > >> > >> I changed var x in both location into smallint and everything worked > >> fine for me. > > > > You are sure that a smallint is large enough in all cases? > > I'm not sure, but according to: > http://www.firebirdsql.org/index.php?op=guide&id=techspec > the approximation of maximum 16,384 integer fields could be allowed per table, > and smallint has a range of -32768..32767, double of whats stated. > Unless someone has more columns of smaller size data. > > Please let me know the ideal type before I prepare the patch.
It's up to you to decide, but as you stated smallint seems ok. > Should the patch be created for trunk or fixes_22 (I was using fixes_22 to > test). In principle all patches should be made against trunk. But in this case I don't suspect many differences so a patch against fixes will probably also work. And else I can convert it. Joost. _______________________________________________ fpc-pascal maillist - fpc-pascal@lists.freepascal.org http://lists.freepascal.org/mailman/listinfo/fpc-pascal
