Fwd: Re: RFR(xxs): 8072935: Fix missing newline at end of file after 8067447

Volker Simonis volker.simonis at gmail.com
Fri Feb 13 11:08:18 UTC 2015

Hi David,

Thomas' initial change which started this discussion, i.e.


adds the missing newline to src/share/vm/services/writeableFlags.cpp.
It should be pushed to jdk9-hs-rt/ (preferably before jdk9-hs-rt gets
pushd to jdk9-hs and jdk9-dev).

For this we need a sponsor. Maybe you can volunteer:)

Independently of this fix I've suggested to add a check for missing
newline at the end of file to jcheck. I've meanwhile opend a
corresponding bug and submitted a patch for review to the
code-tools-dev list:


But again, this will only prevent such errors in the future. The
current file which has no newline at the end has to be fixed with
Thomas' patch.


On Fri, Feb 13, 2015 at 11:09 AM, David Holmes <david.holmes at oracle.com> wrote:
> On 13/02/2015 7:23 PM, Volker Simonis wrote:
>> Hi,
>> could somebody please sponsor and push this change.
> Push it to where, as what? I thought you wanted a jcheck addition.
> David
>> Thanks,
>> Volker
>> On Fri, Feb 13, 2015 at 7:49 AM, Martin Buchholz <martinrb at google.com>
>> wrote:
>>> Source files should have exactly one trailing newline.
>>> find -iregex '.*\.\(java\|txt\|c\|cc\|h\|hpp\|cpp\)$' | xargs perl -0777
>>> -ne
>>> 'print "Must have exactly one trailing newline: $ARGV\n" unless
>>> m~[^\n]\Z~s'

More information about the build-dev mailing list