[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [20240] trunk/blender/intern/ghost/intern/ GHOST_SystemWin32.cpp: Patch #18758 for bug #17423 by Matt D.

Dalai Felinto dfelinto at gmail.com
Mon May 18 22:16:57 CEST 2009


So ? Anyone?
To fix that here I changed:

-if(!lResult)
-   lResult = 0;

by:
+lResult = 0;

Can I commit this?


2009/5/17 Benoit Bolsee <benoit.bolsee at online.be>:
> This patch causes an MSVC uninitialized variable runtime condition with
> the lResult variable at line 911:
>
> if (event) {
>        system->pushEvent(event);
>        if(!lResult)    <<=== here
>                lResult = 0;
> }
>
> Should lResult be initialized?
>
> One remark:     if(!lResult) lResult = 0;    seems pretty much useless.
> What was the intention?
>
>
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers
>


More information about the Bf-committers mailing list