Re: [PATCH 2/3] server: STATUS_MORE_PROCESSING_REQUIRED does not indicate that an async operation is complete (try 4).

[Alexandre Julliard]

"Erich E. Hoover"  writes:
> @@ -285,7 +285,9 @@ void async_set_result( struct object *obj, unsigned int status, unsigned int tot
>          if (async->timeout) remove_timeout_user( async->timeout );
>          async->timeout = NULL;
>          async->status = status;
> -
>    if (async->data.cvalue)
> -
>    if (status == STATUS_MORE_PROCESSING_REQUIRED)
> -
>        async->status = STATUS_PENDING;
> -
>    else if (async->data.cvalue)

Why are you marking the async pending?  That doesn't seem right, it's
terminated at that point.

[PATCH 2/3] server: STATUS_MORE_PROCESSING_REQUIRED does not indicate that an async operation is complete (try 4).

[Erich E. Hoover]

On Wed, Jun 6, 2012 at 5:56 AM, Alexandre Julliard  wrote:
> "Erich E. Hoover"  writes:
> > @@ -285,7 +285,9 @@ void async_set_result( struct object *obj, unsigned int status, unsigned int tot
> >          if (async->timeout) remove_timeout_user( async->timeout );
> >          async->timeout = NULL;
> >          async->status = status;
> > - if (async->data.cvalue)
> > - if (status == STATUS_MORE_PROCESSING_REQUIRED)
> > - async->status = STATUS_PENDING;
> > - else if (async->data.cvalue)
>
> Why are you marking the async pending?  That doesn't seem right, it's
> terminated at that point.

That would be because I misread the MSDN article a little bit (either
that or the one I just looked up is a different one from what I
originally read).  Would you like me to resubmit everything or just
part 2 and 3?

Erich Hoover
[e-mail]