RFR: CODETOOLS-7900964 - [webrev] Option -f does not generate diffs for files whose path happens to contain the name of their repo

Mike Duigou mike.duigou at oracle.com
Sat Jul 12 02:13:30 UTC 2014


Looks fine to me.

Mike

On Jun 26 2014, at 09:42 , Daniel Fuchs <daniel.fuchs at oracle.com> wrote:

> Hi,
> 
> Please find below a patch for:
> 
> CODETOOLS-7900964 - [webrev] Option -f does not generate
>                    diffs for files whose path happens to
>                    contain the name of their repo
> 
> https://bugs.openjdk.java.net/browse/CODETOOLS-7900964
> 
> http://cr.openjdk.java.net/~dfuchs/webrev_CODETOOLS-7900964/webrev.00/
> 
> The change is simple:
> 
> The only case where should *not* print the name of the tree is when
> the index returned by index($1,tree) is 1 - so changing (n == 0) in
> (n != 1) solves the issue.
> 
> best regards,
> 
> -- daniel



More information about the webrev-dev mailing list