Incidencia #43355

zstd printf size_t trouble

Abrir Fecha: 2021-12-09 08:48 Última actualización: 2021-12-09 09:52

Informador:
Propietario:
Tipo:
Estado:
Cerrado
Componente:
Prioridad:
9 - Highest
Gravedad:
5 - Medium
Resolución:
Fixed
Fichero:
2

Details

Build on msys2 currently fails (detected by CI). The problem is bad format specifier for a size_t variable in zstd support code.

Will treat this as an emergency fix, even though there's multiple ways to proceed and all of them have their risks.

The obvious solution is to provide correct "%zu" format specifier, and that's what I'm going to test first. The risk associated to that is that it might not be supported by some of the compilers we've supported so far.

Ticket History (3/7 Histories)

2021-12-09 08:48 Updated by: cazfi
  • New Ticket "zstd printf size_t trouble" created
2021-12-09 08:54 Updated by: cazfi
  • Resolución Update from Ninguno to Accepted
2021-12-09 09:33 Updated by: cazfi
  • Resolución Update from Accepted to Ninguno
Comentario

Reply To cazfi

The obvious solution is to provide correct "%zu" format specifier, and that's what I'm going to test first. The risk associated to that is that it might not be supported by some of the compilers we've supported so far.

Didn't work even in msys2, where the fix was needed for. Trying another approach...

2021-12-09 09:43 Updated by: cazfi
  • Resolución Update from Ninguno to Accepted
Comentario

The new patch casts the zstd error value to a 'long' on all environments, so '%ld' format specifier can be used.

2021-12-09 09:52 Updated by: cazfi
  • Estado Update from Open to Cerrado
  • Propietario Update from (Ninguno) to cazfi
  • Resolución Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Entrar