Not true. It was comparing old and new values of svn:keywords but should compare old and new values of svn:eol-style.
Ok, I got the idea of high-level problem. > On Mon, Jun 25, 2012 at 04:03:42PM +0200, Dmitry Pavlenko wrote: > > I would also ask you to add the fix to 1.7.x > > > > [[[ > > Fix a typo that could lead to wrong translated_size value. > > if svn:eol-style is locally changed and svn:keywords is not, > > translated_size wasn't reset. > > > > * subversion/libsvn_wc/props.c > > > > (do_propset): SVN_PROP_EOL_STYLE value should be checked, not > > SVN_PROP_KEYWORDS. > > > > ]]] > > The fix is correct. I've committed it using the above log message. > > I think it might be better to explain the problem at a higher level in the > first part of the log message. From reading the log message and the code > in do_propset() it's not clear what translated_size is referring to. > > Maybe something like this? > > [[[ > 'svn propset' was comparing the new value of svn:eol-style to the > svn:keywords property, rather than the old value of the svn:eol-style > property. This could prevent an update of cached values related to > EOL expansion in wc.db. > > * subversion/libsvn_wc/props.c > (do_propset): Use SVN_PROP_EOL_STYLE instead of SVN_PROP_KEYWORDS > to obtain the value of the svn:eol-style property from prophash. > ]]] > > > [[[ > > Index: subversion/libsvn_wc/props.c > > =================================================================== > > --- subversion/libsvn_wc/props.c (revision 1353527) > > +++ subversion/libsvn_wc/props.c (working copy) > > @@ -2315,7 +2315,7 @@ do_propset(svn_wc__db_t *db, > > > > } > > > > else if (kind == svn_node_file && strcmp(name, SVN_PROP_EOL_STYLE) == > > 0) > > > > { > > > > - svn_string_t *old_value = apr_hash_get(prophash, > > SVN_PROP_KEYWORDS, + svn_string_t *old_value = > > apr_hash_get(prophash, SVN_PROP_EOL_STYLE, > > > > APR_HASH_KEY_STRING); > > > > if (((value == NULL) != (old_value == NULL)) > > > > ]]]