RFR: 8230117 removing dead code from jar tool

[Adam Sotona]

Hi Christoph,
thank you for clarification, here it is:

JBS:  https://bugs.openjdk.java.net/browse/JDK-8230117 <https://bugs.openjdk.java.net/browse/JDK-8230117>
webrev: http://cr.openjdk.java.net/~asotona/8230117/webrev.00/ <http://cr.openjdk.java.net/~asotona/8230117/webrev.00/>

Mach5 Tier1, Tier2 and Tier3 tests passed.

Thanks,
Adam

> On 1 Mar 2020, at 21:16, Langer, Christoph <christoph.langer at sap.com> wrote:
> 
> Hi Adam, Lance,
> 
> as you've already figured out, attachments sent out to the mailing list will be removed. So it's important to generate webrevs and link to it or, for smaller patches, inline it. ��
> 
> Also, the RFR subject is missing a bug ID. From the URL to the webrev I take the bug ID is JDK-8230117. Going to JBS it seems that the bug is confidential. Could you please either open up the bug before pushing or alternatively, create a new, public bug to push this against?
> 
> Thanks
> Christoph
> 
> 
> 
>> -----Original Message-----
>> From: core-libs-dev <core-libs-dev-bounces at openjdk.java.net> On Behalf
>> Of Lance Andersen
>> Sent: Freitag, 28. Februar 2020 18:04
>> To: Sean Mullan <sean.mullan at oracle.com>
>> Cc: core-libs-dev at openjdk.java.net
>> Subject: Re: RFR: removing dead code from jar tool
>> 
>> sigh it is being removed it appears by the mail server
>> 
>> I placed a copy here: http://cr.openjdk.java.net/~lancea/JDK-8230117.patch
>> <http://cr.openjdk.java.net/~lancea/JDK-8230117.patch>
>> 
>>> On Feb 28, 2020, at 12:00 PM, Lance Andersen
>> <lance.andersen at oracle.com> wrote:
>>> 
>>> Here it is again
>>> 
>>>> On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan at oracle.com>
>> wrote:
>>>> 
>>>> I think you forgot to attach the patch.
>>>> 
>>>> --Sean
>>>> 
>>>> On 2/28/20 2:28 AM, Adam Sotona wrote:
>>>>> Hi,
>>>>> I would like to ask for review of the attached patch removing dead code
>> from jar tool
>>>>> Class files sun.tools.jar.Manifest and sun.tools.jar.SignatureFile appear
>> to be dead code and should be removed.
>>>>> Build with the patch passes all Tier1, Tier2 and Tier3 tests.
>>>>> Thank you,
>>>>> Adam
>>> 
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
>> Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>> Oracle Java Engineering
>>> 1 Network Drive
>>> Burlington, MA 01803
>>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>> 
>>> 
>> 
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen|
>> Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering
>> 1 Network Drive
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>> 
>> 
>