[fpc-devel] Test fix

Pierre Muller pierre at freepascal.org
Fri Apr 25 22:31:54 CEST 2025


Hi,

   I add the IN_TESTS conditional code to avoid getting timeout related failures,
inside testsuite.

   I didn't notice that more than one iteration was needed for bfloatfunc.pp,
because I tried it with ITERATIONS=1 and it seemed to finish without failure...

   I have no objection to your patch, please go ahead and commit.

For tests/bench/bcase.pp, I had to set ITERATIONS To $10000
to avoid errors...

   Just one small remark: why do you use '$8' instead of simply '8'?

Pierre


Le 25/04/2025 à 21:52, J. Gareth Moreton via fpc-devel a écrit :
> Hi everyone,
> 
> I just fixed an issue with tests/bench/bfloatfunc.pp over at
> https://gitlab.com/freepascal.org/fpc/source/-/merge_requests/996
> failing due to the reduced iteration count.  I've increased it from 1 to
> 8 when in conformance mode (in IN_TESTS defined), since if it's less
> than that, not all of the tests get executed and, as a consequence, many
> of the results are uninitialised and cause false failures.
> 
> Just wanted to verify that it's good to merge since I'm reluctant to
> press Merge myself without approval.
> 
> Kit
> 
> 



More information about the fpc-devel mailing list