Actually bug introduced on revision: 19146 (feb 2008).<div>Benoit, I'm pretty confident of the fix. If the original code is the "right one" then we need to edit the console message (since the behaviour was buggy for sure). Let me know otherwise...</div>
<div><br></div><div>Cheers,</div><div>Dalai</div><div><br></div><div><br><div class="gmail_quote">2010/9/12 Dalai Felinto <span dir="ltr"><<a href="mailto:dfelinto@gmail.com">dfelinto@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Revision: 31900<br>
          <a href="http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=31900" target="_blank">http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=31900</a><br>

Author:   dfelinto<br>
Date:     2010-09-13 06:52:20 +0200 (Mon, 13 Sep 2010)<br>
<br>
Log Message:<br>
-----------<br>
blenderplayer commandline argument fixes (for -f and -w)<br>
argc always include the filename, therefore all the optional parameters should check for < argc instead of <=argc.<br>
<br>
E.g. That was causing "blenderplayer -f 800 600 myfile.blend" to play in a wrong  Bpp (since it was trying to parse the filename to fullScreenBpp).<br>
<br>
Bug introduced on rev. 16448 (in 2.49) and merged to 2.5 in rev. 19323<br>
<br>
Modified Paths:<br>
--------------<br>
    trunk/blender/source/gameengine/GamePlayer/ghost/GPG_ghost.cpp<br>
<br>
Modified: trunk/blender/source/gameengine/GamePlayer/ghost/GPG_ghost.cpp<br>
===================================================================<br>
--- trunk/blender/source/gameengine/GamePlayer/ghost/GPG_ghost.cpp      2010-09-13 02:35:59 UTC (rev 31899)<br>
+++ trunk/blender/source/gameengine/GamePlayer/ghost/GPG_ghost.cpp      2010-09-13 04:52:20 UTC (rev 31900)<br>
@@ -479,14 +479,14 @@<br>
                                i++;<br>
                                fullScreen = true;<br>
                                fullScreenParFound = true;<br>
-                               if ((i + 2) <= argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
+                               if ((i + 2) < argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
                                {<br>
                                        fullScreenWidth = atoi(argv[i++]);<br>
                                        fullScreenHeight = atoi(argv[i++]);<br>
-                                       if ((i + 1) <= argc && argv[i][0] != '-')<br>
+                                       if ((i + 1) < argc && argv[i][0] != '-')<br>
                                        {<br>
                                                fullScreenBpp = atoi(argv[i++]);<br>
-                                               if ((i + 1) <= argc && argv[i][0] != '-')<br>
+                                               if ((i + 1) < argc && argv[i][0] != '-')<br>
                                                        fullScreenFrequency = atoi(argv[i++]);<br>
                                        }<br>
                                }<br>
@@ -497,11 +497,11 @@<br>
                                fullScreen = false;<br>
                                windowParFound = true;<br>
<br>
-                               if ((i + 2) <= argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
+                               if ((i + 2) < argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
                                {<br>
                                        windowWidth = atoi(argv[i++]);<br>
                                        windowHeight = atoi(argv[i++]);<br>
-                                       if ((i +2) <= argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
+                                       if ((i + 2) < argc && argv[i][0] != '-' && argv[i+1][0] != '-')<br>
                                        {<br>
                                                windowLeft = atoi(argv[i++]);<br>
                                                windowTop = atoi(argv[i++]);<br>
<br>
<br>
_______________________________________________<br>
Bf-blender-cvs mailing list<br>
<a href="mailto:Bf-blender-cvs@blender.org">Bf-blender-cvs@blender.org</a><br>
<a href="http://lists.blender.org/mailman/listinfo/bf-blender-cvs" target="_blank">http://lists.blender.org/mailman/listinfo/bf-blender-cvs</a><br>
</blockquote></div><br></div>