facebook / facebook/zstd

seekable_format warning with -DNDEBUG: potential uninitialised local variable newOffset used

Offen
#4,756 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C
Sterne
27.9k
Forks
2.6k
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
8

Beschreibung

**Describe the bug**
When seekable_format is compiled with -DNDEBUG (common for release builds), the assert inside the switch is eliminated. As a result the "unreachable" code from lines 190 onwards become reachable and the compiler correctly determines that in line 193 the variable is used uninitialised triggering a warning. Since compiling with DNDEBUG is a valid way to reduce run-time overhead in release environments, I suggest this use-of-uninitialised-variable should be mitigated without the use of assert(), perhaps by `return -1;` in the default case below the assert(0);

**To Reproduce**
Steps to reproduce the behavior:
1. Compile seekable_format with -DNDEBUG.

**Expected behavior**
No warnings are expected during compilation of seekable format or linking of a static library file resulting from such compilation.

**Compilation environment:**
- OS: Windows
- Version 1.5.7, also head of MAIN
- Compiler MSVC 19.29.30159.0
- Flags /W4 /O2 /DNDEBUG
- Build system CMake

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.