Results 293 comments of Samuel Thibault

Please also update the expected output in the test files, see the check failures

It seems to be the unfortunate behavior of `AC_PROG_CXX`. I see a lot of `configure.ac` files that just use it and indeed not actually properly test in that case.

For information, I have submitted as https://github.com/espeak-ng/espeak-ng/pull/1095 what I believe is proper boundary check. Of course dynamic reallocation would allow to avoid cutting speech, but at least: * the patch...

Please explain why this is needed, what it fixes. Also, see the check failure, possibly you have to update the test expected output.

Please see the checks failures

Also, please see the build failure: ``` Using phonemetable: 'ar' Compiling: 'ar_listx' 30089 entries Compiling: 'ar_list' 136: Bad phoneme [I] (U+49) in: _0 s[-'IfR 251 entries Compiling: 'ar_emoji' 1639 entries...

Please see the `make check` errors

https://github.com/espeak-ng/espeak-ng/pull/1091 can be included for a start

#1105 adds an usan profile, it depends on pull requests #1093 #1094 #1095 #1102 #1103 #1104 before we can merge it

#1110 adds an asan profile, it depends on pull requests #1095 #1103 #1106 #1107 #1109 before we can merge it