RE: commit: ldap/servers/slapd abandon.c cancel.c

["Howard Chu" <hyc@highlandsun.com>]

> -----Original Message-----
> From: owner-openldap-commit@OpenLDAP.org
> [mailto:owner-openldap-commit@OpenLDAP.org]On Behalf Of  hyc@OpenLDAP.org

> Update of /repo/OpenLDAP/pkg/ldap/servers/slapd
>
> Modified Files:
> 	abandon.c  1.38 -> 1.39
> 	cancel.c  1.14 -> 1.15
>
> Log Message:
> Fix ITS#3160, decrement conn->c_n_ops_pending when removing an op from
> the conn->c_pending_ops queue.
>
>
> CVS Web URLs:
>   http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/
>     http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/abandon.c
>     http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/cancel.c

It occurs to me that Cancel will never remove a pending operation, because
the Cancel exop itself will be forced onto the pending list whenever the
pending list is non-empty. Is this important?

  -- Howard Chu
  Chief Architect, Symas Corp.       Director, Highland Sun
  http://www.symas.com               http://highlandsun.com/hyc
  Symas: Premier OpenSource Development and Support