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