kc3-lang/SDL

• Show log

  Commit

• Hash : b5ea3c6d
  Author :
  Date : 2017-08-11T21:30:06
  

  Fixed bug 3284 - minor correction for SDL_setenv on _WIN32__ platform
  
  Coriiander
  
  Here is a minor correction for a non-breaking mistake in SDL_setenv for __WIN32__ platform. See below for details.
  
  FILE:
  "SDL/src/stdlib/SDL_getenv.c"
  
  FUNCTION: (__WIN32__ platform)
  int SDL_setenv(const char *name, const char *value, int overwrite)
  
  CODE:
      if (!overwrite) {
          char ch = 0;
          const size_t len = GetEnvironmentVariableA(name, &ch, sizeof (ch));
          if (len > 0) {
              return 0;  /* asked not to overwrite existing value. */
          }
      }
  
  
  WHAT'S WRONG:
  The 3th argument to GetEnvironmentVariable (being DWORD nSize) must be the number of characters, not the number of bytes. SDL currently passes "the size of 1 char", rather "1". While it is non-breaking (1=1 after all), it is incorrect. Furthermore there is no need to specify the 2nd and 3th arguments at all.
  
  CORRECTION 1: (corrected argument_
      if (!overwrite) {
          char ch = 0;
          const size_t len = GetEnvironmentVariableA(name, &ch, 1);
          if (len > 0) {
              return 0;  /* asked not to overwrite existing value. */
          }
      }
  
  CORRECTION 2: (stripped of unneeded code)
      if (!overwrite) {
          if (GetEnvironmentVariableA(name, NULL, 0) > 0) {
              return 0;  /* asked not to overwrite existing value. */
          }
      }
  

• Files

• .hgignore
• Android.mk
• BUGS.txt
• CMakeLists.txt
• COPYING.txt
• CREDITS.txt
• INSTALL.txt
• Makefile.in
• Makefile.minimal
• Makefile.pandora
• Makefile.psp
• Makefile.wiz
• README-SDL.txt
• README.txt
• SDL2.spec.in
• SDL2Config.cmake
• TODO.txt
• VisualC-WinRT/
• VisualC.html
• VisualC/
• WhatsNew.txt
• Xcode-iOS/
• Xcode/
• acinclude/
• android-project/
• autogen.sh
• build-scripts/
• cmake/
• cmake_uninstall.cmake.in
• configure
• configure.in
• debian/
• docs/
• include/
• sdl2-config.cmake.in
• sdl2-config.in
• sdl2.m4
• sdl2.pc.in
• src/
• test/
• visualtest/

• Properties

• Git HTTP	https://git.kmx.io/kc3-lang/SDL.git
  Git SSH	git@git.kmx.io:kc3-lang/SDL.git
  Public access ?	public
  Description	Fork of https://github.com/libsdl-org/SDL
  Users
  Tags