Looks good!
Thanks,
/Staffan
> On 8 dec 2014, at 07:57, joe darcy <joe.da...@oracle.com> wrote:
>
> Hello,
>
> Please review the small patch below to fix
>
> JDK-8066636: Suppress deprecation warnings in the jdk.jvmstat and jdk.jdi
> modules
>
> Thanks,
>
> -Joe
>
> diff -r 913808eaf19a
> src/jdk.jdi/share/classes/com/sun/tools/example/debug/expr/ExpressionParser.java
> ---
> a/src/jdk.jdi/share/classes/com/sun/tools/example/debug/expr/ExpressionParser.java
> Mon Nov 10 08:43:27 2014 -0800
> +++
> b/src/jdk.jdi/share/classes/com/sun/tools/example/debug/expr/ExpressionParser.java
> Sun Dec 07 22:52:50 2014 -0800
> @@ -59,6 +59,7 @@
> StackFrame get() throws IncompatibleThreadStateException;
> }
>
> + @SuppressWarnings("deprecation")
> public static Value evaluate(String expr, VirtualMachine vm,
> GetFrame frameGetter) throws ParseException,
> InvocationException,
> diff -r 913808eaf19a
> src/jdk.jvmstat/share/classes/sun/jvmstat/perfdata/monitor/PerfDataBufferImpl.java
> ---
> a/src/jdk.jvmstat/share/classes/sun/jvmstat/perfdata/monitor/PerfDataBufferImpl.java
> Mon Nov 10 08:43:27 2014 -0800
> +++
> b/src/jdk.jvmstat/share/classes/sun/jvmstat/perfdata/monitor/PerfDataBufferImpl.java
> Sun Dec 07 22:52:50 2014 -0800
> @@ -158,6 +158,7 @@
> * the sun.jvmstat.perfdata.aliasmap file indicates some other
> * file as the source.
> */
> + @SuppressWarnings("deprecation")
> private void buildAliasMap() {
> assert Thread.holdsLock(this);
>
> diff -r 913808eaf19a
> src/jdk.jvmstat/share/classes/sun/tools/jstatd/Jstatd.java
> --- a/src/jdk.jvmstat/share/classes/sun/tools/jstatd/Jstatd.java Mon Nov 10
> 08:43:27 2014 -0800
> +++ b/src/jdk.jvmstat/share/classes/sun/tools/jstatd/Jstatd.java Sun Dec 07
> 22:52:50 2014 -0800
> @@ -76,6 +76,7 @@
> }
> }
>
> + @SuppressWarnings("deprecation") // Use of RMISecurityManager
> public static void main(String[] args) {
> String rminame = null;
> int argc = 0;
>
