On 07.09.2014 00:01, stef...@apache.org wrote: > --- subversion/trunk/subversion/libsvn_fs_fs/rev_file.c (original) > +++ subversion/trunk/subversion/libsvn_fs_fs/rev_file.c Sat Sep 6 22:01:09 > 2014 > @@ -176,6 +176,9 @@ open_pack_or_rev_file(svn_fs_fs__revisio > > /* We failed for the first time. Refresh cache & retry. */ > SVN_ERR(svn_fs_fs__update_min_unpacked_rev(fs, scratch_pool)); > + file->start_revision = rev < ffd->min_unpacked_rev > + ? rev - (rev % ffd->max_files_per_dir) > + : rev;
Even though the precedence rules make this assignment behave as intended, I would really, really prefer to have parentheses around the ternary expression. -- Brane
