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

2016-03-21 Thread Anthony Walter
Maciej, Please let me know when you apply Sven's recommendations to the make commands and I'll test. Thanks.. ___ 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-21 Thread David Jenkins
On 3/21/16 3:44 PM, Michael Van Canneyt wrote: The reason for this behaviour is that find needs to be sure that the list is sorted. If it thinks the list is not sorted, the result cannot be guaranteed to be correct, and this is an error condition. What I don't understand about these

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

2016-03-21 Thread Zoƫ Peterson
On 3/21/2016 3:44 PM, Michael Van Canneyt wrote: The reason for this behaviour is that find needs to be sure that the list is sorted. If it thinks the list is not sorted, the result cannot be guaranteed to be correct, and this is an error condition. Then it should raise an exception, not

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

2016-03-21 Thread Michael Van Canneyt
On Mon, 21 Mar 2016, David Jenkins wrote: On 3/21/16 2:55 PM, Michael Van Canneyt wrote: We can introduce a property which disables the check, if you want. Michael. ___ fpc-devel maillist - fpc-devel@lists.freepascal.org

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

2016-03-21 Thread David Jenkins
On 3/21/16 2:55 PM, Michael Van Canneyt wrote: We can introduce a property which disables the check, if you want. Michael. ___ 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-21 Thread David Jenkins
On 3/21/16 2:55 PM, Michael Van Canneyt wrote: We can introduce a property which disables the check, if you want. Michael. ___ fpc-devel maillist - fpc-devel@lists.freepascal.org http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel That

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

2016-03-21 Thread Michael Van Canneyt
On Mon, 21 Mar 2016, David Jenkins wrote: I just ran across the change that checks Sorted at beginning of find and exits if Sorted = False. This has caused some problems for us a) We have lists that are sorted manually and we'd like to continue keeping them sorted manually (and be able

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

2016-03-21 Thread David Jenkins
I just ran across the change that checks Sorted at beginning of find and exits if Sorted = False. This has caused some problems for us a) We have lists that are sorted manually and we'd like to continue keeping them sorted manually (and be able to call .insert()). This means not setting

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

2016-03-21 Thread Sven Barth
On 21.03.2016 15:21, Maciej Izak wrote: > 2016-03-21 15:08 GMT+01:00 Michael Van Canneyt >: > > No need for that. > > You just need to {$IFNDEF VER3_0} all record operators code. > > A make cycle compiles 3 times. > >

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

2016-03-21 Thread Maciej Izak
2016-03-21 15:08 GMT+01:00 Michael Van Canneyt : > No need for that. > > You just need to {$IFNDEF VER3_0} all record operators code. > > A make cycle compiles 3 times. > > During a make cycle you must start with 3.0.0, this will compile the RTL > with 3.0.0 and this

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

2016-03-21 Thread Michael Van Canneyt
On Mon, 21 Mar 2016, Maciej Izak wrote: 2016-03-21 14:28 GMT+01:00 Sven Barth : The RTL *must* be compileable with the latest release. You need to use ifdefs to ensure that. If a "make all" at the top level with a FPC 3.0.0 does not work then your branch won't

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

2016-03-21 Thread Maciej Izak
2016-03-21 14:28 GMT+01:00 Sven Barth : > The RTL *must* be compileable with the latest release. You need to use > ifdefs to ensure that. > > If a "make all" at the top level with a FPC 3.0.0 does not work then your > branch won't be merged. Simple as that... > So

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

2016-03-21 Thread Sven Barth
Am 21.03.2016 11:51 schrieb "Maciej Izak" : > > 2016-03-21 2:12 GMT+01:00 Anthony Walter : >> >> Just a note, I attempted to compile your smart_pointer branch revision 33229 on a Raspberry Pi 3 Raspbian Jessie (with the latest updates) and the build failed. >>

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

2016-03-21 Thread Maciej Izak
2016-03-21 2:12 GMT+01:00 Anthony Walter : > Just a note, I attempted to compile your smart_pointer branch revision > 33229 on a Raspberry Pi 3 Raspbian Jessie (with the latest updates) and the > build failed. > > make all OPT="-dFPC_ARMHF" > > Sorry, I didn't capture the output