On 22 May 2013 11:20, Stefan Dösinger <stefandoesin...@gmail.com> wrote: > The attached patch passes all the tests without todo_wine. However, > I'm not sure if there are any unintended side effects when I change > the order of the swapchain re-creation and focus window > release/acquire. I'll probably put this into a different patch for > bisecting reasons. > Yeah, makes sense. Applications tend to be pretty sensitive to this kind of stuff.
> It's merely moved from its original position. I'd like to keep it for > consistency as pretty much all ddraw methods have a small > documentation comment. > It's not a big deal, but I generally remove those if I touch the function for some other reason, unless there's information in there that's not immediately obvious from just looking at the function.