Re: [PATCH] Refactor code for --auto-props and --no-auto-props mutual exclusion

Hi,

Karl Fogel wrote:
> declared), but do watch out for tabs -- those should be spaces
> instead, so the indentation will come out right.

I ve corrected it in the new patch.

> Also, why make it conditional on svn_cl__add or svn_cl__import? I
> would think autoprops and no_autoprops would be mutually exclusive in
> any conceivable universe :-). If so, it makes sense to check for the
> exclusivity outside any other conditional, since the check is cheap.

As per the above suggestion by Karl, I am reattaching the new updated
patch along with this mail.

[[[
Follow-up to refactoring r26007

    * subversion/svn/main.c
      (main): Check for sanity of argument combinations only once after
       parsing all the arguments for --auto-props and --no-auto-props.

Patch by: Senthil Kumaran <senthil@collab.net>
Suggested by: kfogel
]]]

PS: As per hwright's comments am including my log message inline :)

Thank You.

--
Senthil Kumaran S
http://www.stylesen.org/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org