RFR(S): 8219142: Make libjimage strncpy uses GCC 8.x friendly

[Mikael Vidstedt]

> On Feb 18, 2019, at 12:03 AM, Alan Bateman <alan.bateman at oracle.com> wrote:
> 
> On 15/02/2019 21:24, Mikael Vidstedt wrote:
>> Please review this change which addresses some warnings generated by GCC 8.2 related to the uses of strncpy in libjimage/imageFile.cpp.
>> 
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8219142
>> Webrev: http://cr.openjdk.java.net/~mikael/webrevs/8219142/webrev.00/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8219142/webrev.00/open/webrev/>
>> 
>> 
>> In addition to feedback on the change itself, I’m taking suggestions on what tests to run. An earlier version of the change passed the typical tier1 testing. I’m going to run tier1 on this version as well, but let me know if there are additional tests I should run.
>> 
> The jrtfs tests cover this area, the path to specify to jtreg is jdk/jdk/internal/jrtfs.

Thanks! I ran the jrtfs tests (all of tier2 even) and they all pass.

> I skimmed through the changes and all red looks good :-)  I assume the bug description can be changed as it's now about removing unused jimage functions rather than changes to works with a newer version of gcc.

Updated the summary to reflect the new charter of the enhancement.

Can I please get a review from somebody in the runtime team as well for the hotspot changes?

Cheers,
Mikael