Merged revision(s) 22031 from trunk/OpenMPT:

[Fix] mpg123: Silence MSVC static analyzer warning. ........ git-svn-id: https://source.openmpt.org/svn/openmpt/branches/OpenMPT-1.30@22033 56274372-70c3-4bfc-bfc3-4c3a0b034d27
OpenMPT · Oct 29, 2024 · e8822f4 · e8822f4
1 parent ecd0a3f
commit e8822f4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/include/mpg123/OpenMPT.txt b/include/mpg123/OpenMPT.txt
@@ -7,6 +7,8 @@ The following changes have been made:
     `OPT_*`.
  *  `ports/Xcode/config.h` has been modified to `#define HAVE_DIRENT_H`.
  *  `src/compat/compat.c` has been modified for DJGPP compatibility.
+ *  `src/compat/compat.c` has been modified to silence MSVC static analyzer
+    warnings.
  *  Modifications are marked by `// OpenMPT` or `/* OpenMPT */`.
  *  Obviously, unnecessary folders and files have been removed.
  *  For building, premake is used to generate Visual Studio project files.

diff --git a/include/mpg123/src/compat/compat.c b/include/mpg123/src/compat/compat.c
@@ -164,8 +164,10 @@ int INT123_compat_fclose(FILE *stream)
 void INT123_compat_binmode(int fd, int enable)
 {
 #if   defined(HAVE__SETMODE)
+	(void)  /* OpenMPT */
 	_setmode(fd, enable ? _O_BINARY : _O_TEXT);
 #elif defined(HAVE_SETMODE)
+	(void)  /* OpenMPT */
 	setmode(fd, enable ? O_BINARY : O_TEXT);
 #endif
 }