Changeset 2962 in vbox for trunk/src/VBox/Frontends/VBoxSDL/VBoxSDLTest.cpp

Timestamp:

May 31, 2007 1:17:41 PM (18 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

21657

Message:

Don't use putenv if possible as setenv is more correct. The reason is that putenv expects a parameter which gets part of the environment. If the passed string is changed, the environment changes as well. Passing an automatic variable to putenv is a bug. Though it isn't wrong to pass a constant since a constant string goes never away. But gcc-4.2 warns about it.

File:

• trunk/src/VBox/Frontends/VBoxSDL/VBoxSDLTest.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VBoxSDL/VBoxSDLTest.cpp

@@ -103 +103 @@
     _putenv("SDL_VIDEO_WINDOW_POS=0,0");
 #else
-    putenv("SDL_VIDEO_WINDOW_POS=0,0");
+    setenv("SDL_VIDEO_WINDOW_POS", "0,0", 1);
 #endif