gdi32: Remove unneeded check. (Coverity)

[EA Durbin]

> From: kai.blin at gmail.com
> To: robertshearman at gmail.com
> Subject: Re: gdi32: Remove unneeded check. (Coverity)
> Date: Wed, 28 Jan 2009 10:32:40 +0100
> CC: wine-devel at winehq.org
> 
> On Tuesday 27 January 2009 18:29:10 Rob Shearman wrote:
> 
> > It doesn't make much sense to leave the lcdfilter variable in the
> > function if you're going to remove the if condition depending on it,
> > since it only has one other use.
> > However, It might have been the intention of the author of this code
> > that this is setting to be changed at compile time (or runtime through
> > a registry tweak) and have the code do the appropriate thing - in that
> > case it would have been better to put the variable at the top of the
> > file.
> 
> I actually have no idea. This was the smallest possible fix that addressed the 
> coverity issue, but I agree that we need to do something with the lcdfilter 
> variable.
> 
> Looking at the commits responsible for the lcdfilter assignment (028617b9) and 
> the if check (45a081f1), I don't see anything that hints at why there's an if 
> check with lcdfilter hardcoded. CCing the original author.
> 
> Cheers,
> Kai
> 
> -- 
> Kai Blin
> WorldForge developer  http://www.worldforge.org/
> Wine developer        http://wiki.winehq.org/KaiBlin
> Samba team member     http://www.samba.org/samba/team/
> --
> Will code for cotton.

_________________________________________________________________
Hotmail® goes where you go. On a PC, on the Web, on your phone. 
http://www.windowslive-hotmail.com/learnmore/versatility.aspx#mobile?ocid=TXT_TAGHM_WL_HM_versatility_121208 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20090130/6aadbdf1/attachment.htm