On Wed, Oct 19, 2011 at 12:44 PM, Stefan Sperling <[email protected]> wrote:
> On Wed, Oct 19, 2011 at 07:27:50PM +0200, Daniel Shahaf wrote:
>> Please revert. The if (1) is there to make merging the revprop-packing
>> branch easier.
I'm happy to revert, but as the current state appears to be temporary,
what's the plan to change it / make it permanent?
>> If you don't like the if (1) line, feel free to delete *just that line*
>> and leave the {} block intact...
It wasn't that line, actually, it was the duplicate return statements
which initially led me to that function.
> I think I also removed this once.
> Can you please add a comment if you intend to keep it?
+1
-Hyrum
--
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/
