RFR: 8174837: Add "since=9" to deprecated ContentSigner and ContentSignerParameters classes

Weijun Wang weijun.wang at oracle.com
Tue Feb 14 02:58:05 UTC 2017


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.
>   */
>
> - at Deprecated
> + at 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.
>   */
> - at Deprecated
> + at Deprecated(since="9")
>  public interface ContentSignerParameters {



More information about the security-dev mailing list