You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by ju...@apache.org on 2012/03/27 15:28:23 UTC

svn commit: r1305833 - /subversion/trunk/subversion/libsvn_client/merge.c

Author: julianfoad
Date: Tue Mar 27 13:28:23 2012
New Revision: 1305833

URL: http://svn.apache.org/viewvc?rev=1305833&view=rev
Log:
* subversion/libsvn_client/merge.c
  (find_unmerged_mergeinfo_subroutine): Add a note about this function's name.

Modified:
    subversion/trunk/subversion/libsvn_client/merge.c

Modified: subversion/trunk/subversion/libsvn_client/merge.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/merge.c?rev=1305833&r1=1305832&r2=1305833&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/merge.c (original)
+++ subversion/trunk/subversion/libsvn_client/merge.c Tue Mar 27 13:28:23 2012
@@ -10039,6 +10039,10 @@ find_youngest_merged_rev(svn_revnum_t *y
  * branch SOURCE_URL@SOURCE_REV.  Extend SOURCE_MERGEINFO, modifying it in
  * place, to include the natural history (implicit mergeinfo) of
  * SOURCE_URL@SOURCE_REV.  ### But make these additions in SCRATCH_POOL.
+ *
+ * ### [JAF] This function is named '..._subroutine' simply because I
+ *     factored it out based on code similarity, without knowing what it's
+ *     purpose is.  We should clarify its purpose and choose a better name.
  */
 static svn_error_t *
 find_unmerged_mergeinfo_subroutine(svn_mergeinfo_t *filtered_mergeinfo_p,