Re: [fpc-devel] about commit r29324

2014-12-28 Thread Michael Van Canneyt
On Sun, 28 Dec 2014, Mattias Gaertner wrote: On Sun, 28 Dec 2014 15:36:49 +0100 (CET) Michael Van Canneyt wrote: [...] People really don't seem to get booleans, and why they scorn "While" is beyond me. All these ugly "break" statements :( You may optimize even more by calculating Max:=pre

Re: [fpc-devel] about commit r29324

2014-12-28 Thread Mattias Gaertner
On Sun, 28 Dec 2014 15:36:49 +0100 (CET) Michael Van Canneyt wrote: >[...] > People really don't seem to get booleans, and why they scorn "While" is > beyond me. All these ugly "break" statements :( > > You may optimize even more by calculating Max:=pred(dims) once and doing > I<=Max; Isn't "

Re: [fpc-devel] about commit r29324

2014-12-28 Thread Michael Van Canneyt
On Sun, 28 Dec 2014, Sergei Gorelkin wrote: 28.12.2014 17:36, Michael Van Canneyt пишет: On Sun, 28 Dec 2014, Mattias Gaertner wrote: Hi, forwarded from Zeljko: It's about fpc issue http://bugs.freepascal.org/view.php?id=26370 It seems that TVariantArrayIterator.AtEnd loops for all Dim

Re: [fpc-devel] about commit r29324

2014-12-28 Thread Sergei Gorelkin
28.12.2014 17:36, Michael Van Canneyt пишет: On Sun, 28 Dec 2014, Mattias Gaertner wrote: Hi, forwarded from Zeljko: It's about fpc issue http://bugs.freepascal.org/view.php?id=26370 It seems that TVariantArrayIterator.AtEnd loops for all Dims for no reason, so have impact on performance.

Re: [fpc-devel] about commit r29324

2014-12-28 Thread Michael Van Canneyt
On Sun, 28 Dec 2014, Mattias Gaertner wrote: Hi, forwarded from Zeljko: It's about fpc issue http://bugs.freepascal.org/view.php?id=26370 It seems that TVariantArrayIterator.AtEnd loops for all Dims for no reason, so have impact on performance. *current implementation* function TVariantAr

[fpc-devel] about commit r29324

2014-12-28 Thread Mattias Gaertner
Hi, forwarded from Zeljko: It's about fpc issue http://bugs.freepascal.org/view.php?id=26370 It seems that TVariantArrayIterator.AtEnd loops for all Dims for no reason, so have impact on performance. *current implementation* function TVariantArrayIterator.AtEnd: Boolean; var i : sizeint;