Re: [PATCH 1/2] server: Don't create new handle when DUP_HANDLE_CLOSE_SOURCE is used if possible (try2)

[Alexandre Julliard]

Piotr Caban <piotr(a)codeweavers.com> writes:
> @@ -890,6 +890,7 @@ struct rawinput_device
>      obj_handle_t handle;       /* duplicated handle in dst process */
>      int          self;         /* is the source the current process? */
>      int          closed;       /* whether the source handle has been closed */
> +    int          reused;       /* whether the source handle has been reused */

I don't think you need an extra flag, there's nothing the client has to
do differently in that case (the warning is mostly cosmetic anyway).

-- 
Alexandre Julliard
julliard(a)winehq.org

[PATCH 1/2] server: Don't create new handle when DUP_HANDLE_CLOSE_SOURCE is used if possible (try2)

[Piotr Caban]

On 06/04/13 21:56, Alexandre Julliard wrote:
> Piotr Caban <piotr(a)codeweavers.com> writes:
> > @@ -890,6 +890,7 @@ struct rawinput_device
> >       obj_handle_t handle;       /* duplicated handle in dst process */
> >       int          self;         /* is the source the current process? */
> >       int          closed;       /* whether the source handle has been closed */
> > +    int          reused;       /* whether the source handle has been reused */
>
> I don't think you need an extra flag, there's nothing the client has to
> do differently in that case (the warning is mostly cosmetic anyway).

I'll send a patch with the warning message removed unless you think that 
it's better to keep it.

Thanks,
Piotr