Re: user32: Set correct caret state in the server in SetCaretPos. (try 5)

[Sebastian Lackner]

On 31.12.2015 17:39, Sebastian Lackner wrote:
> From: Anton Baskanov <baskanov(a)gmail.com>
>
> Signed-off-by: Anton Baskanov <baskanov(a)gmail.com>
> Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de>
> ---
>
> Changes since v4:
>
> * Replace !!req->state with (req->state != CARET_STATE_OFF).
>   The first version still depends on CARET_STATE_OFF == 0, which
>   might change in the future.
>
> * Avoid nested ifs by reordering checks.
>
> dlls/user32/caret.c |    8 ++++----
>  server/protocol.def |    6 +++++-
>  server/queue.c      |    8 ++++++--
>  3 files changed, 15 insertions(+), 7 deletions(-)

Any remarks on this patch?