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
>
> Signed-off-by: Anton Baskanov 
> Signed-off-by: Sebastian Lackner
>
> 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?