Hi, On 2024-05-03 22:26, Vincent Bernat wrote: > Hello Helge, > > Couldn't the patch be pushed upstream? And maybe there should be an else > branch with the encoding of the other NaN? >
I am not sure the patch is upstreamable. It basically disables the test on mips and hppa, which is fine for Debian given it is not a regression compared to the previous version where the testsuite was fully disabled. But for upstream, it just hides the real bug. On those architectures, the NaN encoding is indeed different, but the resulted encoded data should be the identical, as defined in the RFC. Therefore I believe the real fix is to convert NaNs (and probably also infinities) during the encoding process. Regards Aurelien -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurel...@aurel32.net http://aurel32.net
