Bug in fastMath.cpp

[dsyme]

The pow10 lookup table has an incorrect duplicate entry on line 19 of fastMath.cpp

This was discovered when experimenting with AI automated test coverage improvement and performance optimization of the code in this repo using GitHub Agentic Workflows.

See https://github.com/dsyme/librapid/pull/7/files#diff-c086ffebf40f52b58e36f5f05414be9084772d011a070efa1f86e010078f3be0R19

The rest of the PR may be of interest but I'm not certain of it so won't submit it.

[Pencilcaseman]

Just pushed a fix -- cheers for finding this.

Regarding that PR, is that actually faster? Do you have benchmark results? That's really cool if it gives better results

[dsyme]

Regarding that PR, is that actually faster? Do you have benchmark results? That's really cool if it gives better results

@Pencilcaseman - not sure yet unfortunately. You're the first repo we're trying this on, still wet paint. Currently trying to get the AI to actually do the work to do the benchmark comparisons. Benchmarking in Actions is ofc not reliable. But maybe enough to get instruction/operation counts or approximate times.

[dsyme]

In this next one the AI claims to have improved performance. Code would need careful checking and added test coverage to match (that can be arranged easily enough)

dsyme#13