[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