webrev -f not working?

[Daniel Fuchs]

Hi Magnus,

Here is a small testcase:

$ echo '[blah/blah]' | gawk '{print substr($1,2,length($1)-1);}'
blah/blah]
$ echo '[blah/blah]' | gawk '{print substr($1,2,length($1)-2);}'
blah/blah

I suspect you won't get the same result on your machine :-(

I wonder what could be causing the difference?

best regards,

-- daniel


On 05/11/14 15:28, Daniel Fuchs wrote:
> On 05/11/14 14:57, Magnus Ihse Bursie wrote:
>> When reading the code, I can't really understand why the old code could
>> ever have worked on a output like that.
>
> Hi Magnus,
>
> If I apply your patch on my machine then I get things like:
>
> jdk]/src/java.logging/share/classes/java/util/logging/LogManager.java
>     ^
> notice the closing bracket.
>
> I wonder - could that be a cr versus crlf thing?
>
> best regards
>
> -- daniel