seekable_format warning with -DNDEBUG: potential uninitialised local variable newOffset used
- 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
Bewertung
Dieses Issue wurde noch nicht bewertet.