Re: [ros-dev] [ros-diffs] [jimtabor] 34276: GetLastActivePopup is still unimplemented, we return the current window handle if it is valid.

[Aleksey Bragin]

It would make sense to add a WARN there or something like that, so it  
could be seen that it is halfplemented only.

WBR,
Aleksey.


On Jul 3, 2008, at 11:17 AM, jimtabor(a)svn.reactos.org wrote:

>   Author: jimtabor
>  Date: Thu Jul  3 02:17:32 2008
>  New Revision: 34276
>
>  URL: http://svn.reactos.org/svn/reactos?rev=34276&view=rev
>  Log:
>  GetLastActivePopup is still unimplemented, we return the current  
>  window handle if it is valid.
>
>  Modified:
>      trunk/reactos/dll/win32/user32/windows/window.c
>
>  Modified: trunk/reactos/dll/win32/user32/windows/window.c
>  URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/ 
>  user32/windows/window.c?rev=34276&r1=34275&r2=34276&view=diff
>  ====================================================================== 
>  ========
>  --- trunk/reactos/dll/win32/user32/windows/window.c [iso-8859-1]  
>  (original)
>  +++ trunk/reactos/dll/win32/user32/windows/window.c [iso-8859-1]  
>  Thu Jul  3 02:17:32 2008
>  @@ -921,8 +921,24 @@
>   HWND STDCALL
>   GetLastActivePopup(HWND hWnd)
>   {
>  -  UNIMPLEMENTED;
>  -  return 0;
>  +    PWINDOW Wnd, WndParent;
>  +    HWND Ret = NULL;
>  +
>  +    Wnd = ValidateHwnd(hWnd);
>  +    if (Wnd != NULL)
>  +    {
>  +        _SEH_TRY
>  +        {
>  +            WndParent = NULL;
>  +            Ret = hWnd;
>  +        }
>  +        _SEH_HANDLE
>  +        {
>  +            /* Do nothing */
>  +        }
>  +        _SEH_END;
>  +    }
>  +    return Ret;
>   }
>
>
>