You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Florian Weimer <fw...@deneb.enyo.de> on 2004/02/06 08:12:45 UTC

include/svn_wc.h refers to obsolete svn_wc_statuses()

Last summer, svn_wc_statuses() was removed and replaced by
svn_wc_get_status_editor(), but the doc comment was not updated.

Index: subversion/include/svn_wc.h
===================================================================
--- subversion/include/svn_wc.h	(revision 8550)
+++ subversion/include/svn_wc.h	(working copy)
@@ -986,14 +986,12 @@
  * become cumbersome: you'd have to roll through a hash to find one
  * lone status.
  * 
- * So we have @c svn_wc_status() for depth 0, and @c svn_wc_statuses() for
- * depths 1 and 2, since the latter two involve multiple return
- * values.
+ * So we have @c svn_wc_status() for depth 0, and @c
+ * svn_wc_get_status_editor() for depths 1 and 2, since the latter two
+ * involve multiple return values.
  *
- * NOTE:  Status structures returned by @c svn_wc_status() or found in
- * the hash created by @c svn_wc_statuses() may contain a @c NULL ->entry
- * field.  This indicates an item that is not versioned in the working
- * copy.
+ * NOTE:  The status structures may contain a @c NULL ->entry field.
+ * This indicates an item that is not versioned in the working copy.
  */
 
 enum svn_wc_status_kind

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: include/svn_wc.h refers to obsolete svn_wc_statuses()

Posted by kf...@collab.net.
Florian, can you repost this (and any other patches) with "[PATCH] "
starting out the Subject line?  That helps our patch manager, Sander
Roobol, notice them and record them in the issue tracker if they don't
get applied quickly (sometimes they do, sometimes they don't).

It also helps if you write a log message (see HACKING) for each patch,
though it's not a big deal -- in these cases, the description in your
email and the patch itself is enough to figure out what's going on.

Thanks,
-Karl

Florian Weimer <fw...@deneb.enyo.de> writes:
> Last summer, svn_wc_statuses() was removed and replaced by
> svn_wc_get_status_editor(), but the doc comment was not updated.
> 
> Index: subversion/include/svn_wc.h
> ===================================================================
> --- subversion/include/svn_wc.h	(revision 8550)
> +++ subversion/include/svn_wc.h	(working copy)
> @@ -986,14 +986,12 @@
>   * become cumbersome: you'd have to roll through a hash to find one
>   * lone status.
>   * 
> - * So we have @c svn_wc_status() for depth 0, and @c svn_wc_statuses() for
> - * depths 1 and 2, since the latter two involve multiple return
> - * values.
> + * So we have @c svn_wc_status() for depth 0, and @c
> + * svn_wc_get_status_editor() for depths 1 and 2, since the latter two
> + * involve multiple return values.
>   *
> - * NOTE:  Status structures returned by @c svn_wc_status() or found in
> - * the hash created by @c svn_wc_statuses() may contain a @c NULL ->entry
> - * field.  This indicates an item that is not versioned in the working
> - * copy.
> + * NOTE:  The status structures may contain a @c NULL ->entry field.
> + * This indicates an item that is not versioned in the working copy.
>   */
>  
>  enum svn_wc_status_kind
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: include/svn_wc.h refers to obsolete svn_wc_statuses()

Posted by kf...@collab.net.
Okay, applied this one in r8573.  Thanks!

Florian Weimer <fw...@deneb.enyo.de> writes:
> Last summer, svn_wc_statuses() was removed and replaced by
> svn_wc_get_status_editor(), but the doc comment was not updated.
> 
> Index: subversion/include/svn_wc.h
> ===================================================================
> --- subversion/include/svn_wc.h	(revision 8550)
> +++ subversion/include/svn_wc.h	(working copy)
> @@ -986,14 +986,12 @@
>   * become cumbersome: you'd have to roll through a hash to find one
>   * lone status.
>   * 
> - * So we have @c svn_wc_status() for depth 0, and @c svn_wc_statuses() for
> - * depths 1 and 2, since the latter two involve multiple return
> - * values.
> + * So we have @c svn_wc_status() for depth 0, and @c
> + * svn_wc_get_status_editor() for depths 1 and 2, since the latter two
> + * involve multiple return values.
>   *
> - * NOTE:  Status structures returned by @c svn_wc_status() or found in
> - * the hash created by @c svn_wc_statuses() may contain a @c NULL ->entry
> - * field.  This indicates an item that is not versioned in the working
> - * copy.
> + * NOTE:  The status structures may contain a @c NULL ->entry field.
> + * This indicates an item that is not versioned in the working copy.
>   */
>  
>  enum svn_wc_status_kind
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org