On Thu, Mar 11, 2010 at 08:52, <ne...@apache.org> wrote: >... > +++ subversion/trunk/subversion/libsvn_wc/adm_ops.c Thu Mar 11 13:52:15 2010 >... > @@ -2219,9 +2229,32 @@ svn_wc__get_pristine_contents(svn_stream > return SVN_NO_ERROR; > } > } > + else > + if (status == svn_wc__db_status_base_deleted)
Woah. This formatting is incorrect. We always do "else if (...". The code above almost makes it look like the "if" is totally separate since it is at the same indentation level, but it ISN'T. The else dramatically changes the meaning. The above style is used nowhere else in our code. Please fix the several uses in this function. >... Cheers, -g
