On 9-8-2011 16:42, Ludo Brands wrote:
> 
> Downloaded the code and it raises an SUnsupportedFieldType when
> creating the ftAutoInc field. That is OK since I didn't include
> ftAutoinc in the supported list for TBufDataset. If you are not
> getting this error, then the patch and/or install didn't work.
I feared as much. Totally strange, as I tried my patch for SQLExport as
well and that did work fine.
I'll clean up everything & try again...

> Line 547 is where FillTestData is called. Not very usefull.
I agree. Seems like the test setup & teardown should be a test of their
own for TBufDataset :(
> 
> I do get a segfault in the last test when assigning to
> ftWideString256. When tracing the code it crashes when moving the
> size+1 chracters to the database. This fixed length moving is
> fundamentally wrong. The string is only 20 or so chars long and
> TCustomBufDataset.SetFieldData tries to do a move of the full 514
> bytes (256+1)*2. This asking for trouble. The source of the data is
> here a temporary WideString created by casting a string, so probably
> on the heap. Reading 514 bytes when only 40 odd where allocated is
> not always working....
> 
> I'll try to fix that one also.
I'd appreciate that a lot!


Thanks a lot for the help,
Reinier
_______________________________________________
fpc-pascal maillist  -  fpc-pascal@lists.freepascal.org
http://lists.freepascal.org/mailman/listinfo/fpc-pascal

Reply via email to