Change looks fine. Thanks Max
On 02/14/2017 12:25 AM, Sean Mullan wrote:
Could I get a quick code review for this simple fix for https://bugs.openjdk.java.net/browse/JDK-8174837?: diff --git a/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSigner.java b/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSigner.java --- a/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSigner.java +++ b/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSigner.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2015, 2017 Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -38,7 +38,7 @@ * @deprecated This class has been deprecated. */ -@Deprecated +@Deprecated(since="9") public abstract class ContentSigner { /** diff --git a/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSignerParameters.java b/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSignerParameters.java --- a/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSignerParameters.java +++ b/src/jdk.jartool/share/classes/com/sun/jarsigner/ContentSignerParameters.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2003, 2015, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -36,7 +36,7 @@ * @author Vincent Ryan * @deprecated This class has been deprecated. */ -@Deprecated +@Deprecated(since="9") public interface ContentSignerParameters {
