[comdlg32] Remove unused variable (Coverity)

Paul Vriens paul.vriens.wine at gmail.com
Sun Nov 12 11:58:06 CST 2006


On 11/12/06, Paul Vriens <paul.vriens.wine at gmail.com> wrote:
>
> Hi,
>
> hResource16 is not used.
>
> Will fix Coverity-17.
>
> Changelog
>   Remove unused variable
>
> Cheers,
>
> Paul.
> ---
> dlls/comdlg32/fontdlg16.c |    3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/comdlg32/fontdlg16.c b/dlls/comdlg32/fontdlg16.c
> index 4b13c2d..c5f11df 100644
> --- a/dlls/comdlg32/fontdlg16.c
> +++ b/dlls/comdlg32/fontdlg16.c
> @@ -164,7 +164,7 @@ INT16 WINAPI FontStyleEnumProc16( SEGPTR
> BOOL16 WINAPI ChooseFont16(LPCHOOSEFONT16 lpChFont)
> {
>      HINSTANCE16 hInst;
> -    HANDLE16 hDlgTmpl16 = 0, hResource16 = 0;
> +    HANDLE16 hDlgTmpl16 = 0;
>      HGLOBAL16 hGlobal16 = 0;
>      BOOL16 bRet = FALSE;
>      LPVOID template;
> @@ -255,7 +255,6 @@ BOOL16 WINAPI ChooseFont16(LPCHOOSEFONT1
>      hInst = GetWindowLongPtrA(HWND_32(lpChFont->hwndOwner),
> GWLP_HINSTANCE);
>      bRet = DialogBoxIndirectParam16(hInst, hDlgTmpl16,
> lpChFont->hwndOwner,
>                       (DLGPROC16) ptr, (DWORD)lpChFont);
> -    if (hResource16) FreeResource16(hDlgTmpl16);
>      if (hGlobal16)
>      {
>          GlobalUnlock16(hGlobal16);
> --
> 1.4.3.5


Forget this one as I'm not sure whether hDlgTmpl16 needs to be freed.

Cheers,

Paul.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20061112/1ccd06ec/attachment.htm


More information about the wine-devel mailing list