Thanks!
I will add additional spaces after commas before pushing the changeset.
Mike
On Apr 16 2014, at 00:24 , Magnus Ihse Bursie <magnus.ihse.bur...@oracle.com>
wrote:
> Fix looks good to me.
>
> Noticed this:
>> while ((blocklen = fread(block,1,sizeof(block),atin)) > 0) {
> Maybe add some space after the commas, if you are fixing other formatting
> issues?
>
> /Magnus
>
> On 15 apr 2014, at 22:30, Mike Duigou <mike.dui...@oracle.com> wrote:
>
>> [fix missing title]
>>
>> On Apr 15 2014, at 13:30 , Mike Duigou <mike.dui...@oracle.com> wrote:
>>
>>> Hello all;
>>>
>>> The recent change to fixpath in JDK-8039411
>>> (https://bugs.openjdk.java.net/browse/JDK-8039411)
>>> (http://hg.openjdk.java.net/jdk9/dev/rev/45183b39d300) introduced a
>>> regression for zero length arguments.
>>>
>>> This changes forces quoting of zero length arguments. It also contains
>>> fixes to a spelling error and cleans up some inconsistent formatting.
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8040332
>>> http://cr.openjdk.java.net/~mduigou/JDK-8040332/0/webrev/
>>>
>>> Mike
>>
