facebook / facebook/zstd

Disabled assert() could use its argument to avoid unused variable warnings

Abierto
#2,868 1 comentario 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
C
Estrellas
27.9k
Forks
2.6k
Merge medio
1 d 3 h
PR fusionados (30 d)
8

Descripción

Test this construction for disabling assertions:

```c
#define assert(condition) \
do { \
if (0) { \
(void)(condition); \
} \
} while (0)
```

Additionally, do the same for `RAWLOG()` and `DEBUGLOG()` when `DEBUGLEVEL < 2`.

This would avoid unused variable warnings that show up in production.

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.