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 2011/12/10 17:12:14 UTC
svn commit: r1212817 - /subversion/trunk/subversion/libsvn_client/mergeinfo.c
Author: julianfoad
Date: Sat Dec 10 16:12:14 2011
New Revision: 1212817
URL: http://svn.apache.org/viewvc?rev=1212817&view=rev
Log:
* subversion/libsvn_client/mergeinfo.c
(filter_log_entry_with_rangelist): Add a question and move two variables
into a more local scope.
Modified:
subversion/trunk/subversion/libsvn_client/mergeinfo.c
Modified: subversion/trunk/subversion/libsvn_client/mergeinfo.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/mergeinfo.c?rev=1212817&r1=1212816&r2=1212817&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/mergeinfo.c (original)
+++ subversion/trunk/subversion/libsvn_client/mergeinfo.c Sat Dec 10 16:12:14 2011
@@ -1346,11 +1346,12 @@ filter_log_entry_with_rangelist(void *ba
/* If the paths changed by LOG_ENTRY->REVISION are provided we can determine
if LOG_ENTRY->REVISION, while only partially represented in
- BATON->RANGELIST, is in fact completely applied to all affected paths. */
+ BATON->RANGELIST, is in fact completely applied to all affected paths.
+ ### And ... what if it is, or if it isn't? What do we do with the answer?
+ And how do we cope if the changed paths are not provided? */
if ((log_entry->non_inheritable || !fleb->filtering_merged)
&& log_entry->changed_paths2)
{
- int i;
apr_hash_index_t *hi;
svn_boolean_t all_subtrees_have_this_rev = TRUE;
apr_array_header_t *this_rev_rangelist =
@@ -1362,11 +1363,11 @@ filter_log_entry_with_rangelist(void *ba
hi;
hi = apr_hash_next(hi))
{
+ int i;
const char *path = svn__apr_hash_index_key(hi);
svn_log_changed_path2_t *change = svn__apr_hash_index_val(hi);
const char *target_fspath_affected;
svn_mergeinfo_t nearest_ancestor_mergeinfo;
- apr_hash_index_t *hi2;
svn_boolean_t found_this_revision = FALSE;
const char *merge_source_rel_target;
const char *merge_source_fspath;
@@ -1438,6 +1439,8 @@ filter_log_entry_with_rangelist(void *ba
if (nearest_ancestor_mergeinfo)
{
+ apr_hash_index_t *hi2;
+
for (hi2 = apr_hash_first(iterpool, nearest_ancestor_mergeinfo);
hi2;
hi2 = apr_hash_next(hi2))