[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#4590) fix ldap_abandon{,ext} man page to match current reality



fenlason@redhat.com wrote:
> Here's a small patch to add a bit more useful verbosity to the
> ldap_abandon man page.
>
> For additional man pages, should I open new bug reports for each, mail
> you the diffs, or something else?
>   

For larger diffs you should probably submit one ITS per diff. (Depending 
on how much is changed, how much review it will require.) If all of the 
diffs are similar enough, you could probably send them in one ITS.

Note in this patch,  "parameter" is misspelled multiple times.
> 			-- JF
>
> Index: ldap_abandon.3
> ===================================================================
> RCS file: /repo/OpenLDAP/pkg/ldap/doc/man/man3/ldap_abandon.3,v
> retrieving revision 1.23
> diff -u -r1.23 ldap_abandon.3
> --- ldap_abandon.3	13 Jun 2006 22:19:32 -0000	1.23
> +++ ldap_abandon.3	15 Jun 2006 19:16:24 -0000
> @@ -43,11 +43,12 @@
>  .I sctrls
>  and
>  .I cctrls
> -parameters, respectively.
> +parameters, respectively.  Either controls paramater may be NULL if no
> +controls of that type are needed.
>  .SH ERRORS
>  .B ldap_abandon_ext()
> -returns an LDAP error code indicating success or failure in sending
> -the Abandon request.
> +returns an LDAP error code indicating success (LDAP_SUCCESS) or failure
> +in sending the Abandon request.
>  .LP
>  See
>  .BR ldap_error (3)
> @@ -57,7 +58,8 @@
>  .B ldap_abandon()
>  routine is deprecated in favor of the
>  .B ldap_abandon_ext()
> -routine. 
> +routine.  It did not have the controls paramaters, and returned a simple
> +0/-1 on success/failure.
>  .LP
>  .so Deprecated
>  
>
>
>   


-- 
  -- Howard Chu
  Chief Architect, Symas Corp.  http://www.symas.com
  Director, Highland Sun        http://highlandsun.com/hyc
  OpenLDAP Core Team            http://www.openldap.org/project/