RFR: removing dead code from jar tool
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
I think this looks OK. I did a search and did not find any usages so I suspect this should be safe to do. Best Lancee
On Feb 28, 2020, at 5:28 AM, Adam Sotona <adam.sotona@oracle.com> 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@oracle.com <mailto:Lance.Andersen@oracle.com>
Thank you, Adam
On 28 Feb 2020, at 12:57, Lance Andersen <lance.andersen@oracle.com> wrote:
I think this looks OK. I did a search and did not find any usages so I suspect this should be safe to do.
Best Lancee
On Feb 28, 2020, at 5:28 AM, Adam Sotona <adam.sotona@oracle.com <mailto:adam.sotona@oracle.com>> 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
<oracle_sig_logo.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> <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@oracle.com <mailto:Lance.Andersen@oracle.com>
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
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@oracle.com>
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@oracle.com> wrote:
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
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@openjdk.java.net> On Behalf Of Lance Andersen Sent: Freitag, 28. Februar 2020 18:04 To: Sean Mullan <sean.mullan@oracle.com> Cc: core-libs-dev@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@oracle.com> wrote:
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
Hi Christoph, Let me sort this out and once I do I will push the change for Adam. Best lance
On Mar 1, 2020, at 3:16 PM, Langer, Christoph <christoph.langer@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@openjdk.java.net> On Behalf Of Lance Andersen Sent: Freitag, 28. Februar 2020 18:04 To: Sean Mullan <sean.mullan@oracle.com> Cc: core-libs-dev@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@oracle.com> wrote:
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
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@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@openjdk.java.net> On Behalf Of Lance Andersen Sent: Freitag, 28. Februar 2020 18:04 To: Sean Mullan <sean.mullan@oracle.com> Cc: core-libs-dev@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@oracle.com> wrote:
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
Hi Adam, thanks for opening up the bug and reposting. Looks good to me ๐ Cheers Christoph From: Adam Sotona <adam.sotona@oracle.com> Sent: Mittwoch, 11. Mรคrz 2020 11:32 To: Langer, Christoph <christoph.langer@sap.com> Cc: Lance Andersen <lance.andersen@oracle.com>; Sean Mullan <sean.mullan@oracle.com>; core-libs-dev@openjdk.java.net Subject: Re: RFR: 8230117 removing dead code from jar tool Hi Christoph, thank you for clarification, here it is: JBS: https://bugs.openjdk.java.net/browse/JDK-8230117 webrev: 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@sap.com<mailto:christoph.langer@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@openjdk.java.net<mailto:core-libs-dev-bounces@openjdk.java.net>> On Behalf Of Lance Andersen Sent: Freitag, 28. Februar 2020 18:04 To: Sean Mullan <sean.mullan@oracle.com<mailto:sean.mullan@oracle.com>> Cc: core-libs-dev@openjdk.java.net<mailto:core-libs-dev@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@oracle.com<mailto:lance.andersen@oracle.com>> wrote: Here it is again On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@oracle.com<mailto:sean.mullan@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@oracle.com<mailto:Lance.Andersen@oracle.com> <mailto:Lance.Andersen@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@oracle.com<mailto:Lance.Andersen@oracle.com> <mailto:Lance.Andersen@oracle.com>
FYI I will be pushing today after one last Mach 5 run completes -- Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen@oracle.com
On Mar 11, 2020, at 8:16 AM, Langer, Christoph <christoph.langer@sap.com> wrote:
๏ปฟ Hi Adam,
thanks for opening up the bug and reposting. Looks good to me ๐
Cheers Christoph
From: Adam Sotona <adam.sotona@oracle.com> Sent: Mittwoch, 11. Mรคrz 2020 11:32 To: Langer, Christoph <christoph.langer@sap.com> Cc: Lance Andersen <lance.andersen@oracle.com>; Sean Mullan <sean.mullan@oracle.com>; core-libs-dev@openjdk.java.net Subject: Re: RFR: 8230117 removing dead code from jar tool
Hi Christoph, thank you for clarification, here it is:
JBS: https://bugs.openjdk.java.net/browse/JDK-8230117 webrev: 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@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@openjdk.java.net> On Behalf Of Lance Andersen Sent: Freitag, 28. Februar 2020 18:04 To: Sean Mullan <sean.mullan@oracle.com> Cc: core-libs-dev@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@oracle.com> wrote:
Here it is again
On Feb 28, 2020, at 11:07 AM, Sean Mullan <sean.mullan@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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
participants (4)
-
Adam Sotona
-
Lance Andersen
-
Langer, Christoph
-
Sean Mullan