[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [2/2] user32: Make a SetFocus message test for a top level
From:       Dmitry Timoshkov <dmitry () baikal ! ru>
Date:       2011-12-30 11:01:08
Message-ID: 20111230195737.48982572.dmitry () baikal ! ru
[Download RAW message or body]

Alexandre Julliard <julliard@winehq.org> wrote:

> > -    if (!set_active_window( hwnd, &prev, FALSE, TRUE )) return 0;
> > +    set_active_window( hwnd, &prev, FALSE, TRUE );
> >      return prev;
> >  }
> 
> It doesn't seem right to ignore all errors. This would need some more
> tests.

What kind of tests do you have in mind? Invalid window? Something else?

-- 
Dmitry.


[prev in list] [next in list] [prev in thread] [next in thread]