RFR: 7141675 Fix jcheck issues in .m sources
    Weijun Wang 
    weijun.wang at oracle.com
       
    Wed Feb  1 05:32:47 PST 2012
    
    
  
I'm wondering if jcheck is updated to include these .m files, will it 
reject the old changesets where the .m files contain trailing 
whitespaces? Or, should those changesets be added into the whitelist?
Thanks
Max
On 02/01/2012 08:25 PM, Anthony Petrov wrote:
> Hi Michael,
>
> The patch looks fine to me. It would be great to update jcheck as well
> so that the formatting requirements be enforced for further putbacks.
>
> --
> best regards,
> Anthony
>
> On 2/1/2012 3:56 PM, Michael McMahon wrote:
>> This change is just to fix some white-space/tab problems that crept into
>> some objective C files whose filename (extension .m) is not currently
>> known to jcheck.
>>
>> http://cr.openjdk.java.net/~michaelm/7141675/webrev.1/
>>
>> There is nothing to see in the webrev as white-space diferences are
>> ignored.
>> But the patch shows the actual changes.
>>
>> Thanks
>> Michael.
    
    
More information about the macosx-port-dev
mailing list