Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Maciej Izak
2016-03-30 7:49 GMT+02:00 Anthony Walter : > I put up a testing page for your modifications here: > > http://www.getlazarus.org/language/smart/ > > The tests are rather simple and sloppy, but so far the few tests I've > written have passed using revision 33384. If anyone wants to add/change the >

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Anthony Walter
I put up a testing page for your modifications here: http://www.getlazarus.org/language/smart/ The tests are rather simple and sloppy, but so far the few tests I've written have passed using revision 33384. If anyone wants to add/change the test source code you can use the edit feature on that pa

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Maciej Izak
2016-03-30 0:06 GMT+02:00 Anthony Walter : > Should I file mantis bug reports for personal branches such as this or is > mailing list notification sufficient? > Mailing list notification looks good for now. Btw. I don't have developer status on FPC mantis part (only for Lazarus part for Sparta p

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Maciej Izak
2016-03-29 19:36 GMT+02:00 Anthony Walter : > TFoo.Initialize is never invoked. > Thanks for report. Fixed at revision: 33384. Important note: Copy(Foos) operation for dyn arrays might be a little confusing (since there is no operator "Copy"). Operator "Finalize" can be called more times than "

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Anthony Walter
Should I file mantis bug reports for personal branches such as this or is mailing list notification sufficient? ___ fpc-devel maillist - fpc-devel@lists.freepascal.org http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel

Re: [fpc-devel] Questions on TStringList.Find change (Mantis 28744)

2016-03-29 Thread David Jenkins
On 3/26/16 4:19 AM, Michael Van Canneyt wrote: I have implemented a new property TStringsSortStyle = (sslNone,sslUser,sslAuto); Property SortStyle : TStringsSortStyle Read FSortStyle Write SetSortStyle; I assume the meaning is clear. - Sorted is true if SortStyle is in [sslUser,sslAuto

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Anthony Walter
Maceij, I did a build using your revision 33381. Compilation with "make all" worked with fpc 3.0.0 as a starting compiler. However my first test with it revealed a pretty serious bug: program mytest; {$mode delphi} type TFoo = record private class operator Initialize(var aFoo: TFoo);

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Sven Barth
Am 29.03.2016 12:36 schrieb "Maciej Izak" : > > 2016-03-29 12:23 GMT+02:00 Sven Barth : >> >> Strange, because for me a "make clean all" tends to work without problems (at least if I haven't messed things up in the compiler myself :P ) > > Nothing strange, when you do forbidden things, like install

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Maciej Izak
2016-03-29 12:23 GMT+02:00 Sven Barth : > Strange, because for me a "make clean all" tends to work without problems > (at least if I haven't messed things up in the compiler myself :P ) > Nothing strange, when you do forbidden things, like installing RTL inside branch folder :P -- Best regards,

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Sven Barth
Am 29.03.2016 12:11 schrieb "Maciej Izak" : > > Done in rev 33381... Anthony, feel free to checkout smart_pointers branch. > > Thanks again Michael and Sven for the tips! That will be useful for further work. I've decide to use {$if FPC_FULLVERSION>30100} instead of {$IFNDEF VER3_0}. > > The last t

Re: [fpc-devel] Feature announcement: Record management operators

2016-03-29 Thread Maciej Izak
Done in rev 33381... Anthony, feel free to checkout smart_pointers branch. Thanks again Michael and Sven for the tips! That will be useful for further work. I've decide to use {$if FPC_FULLVERSION>30100} instead of {$IFNDEF VER3_0}. The last task "make all" with FPC 3.0.0 as start compiler, was d