[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: JDK-8215571: jdb does not include jdk.* in the default class filter
From:       Chris Plummer <chris.plummer () oracle ! com>
Date:       2018-12-20 15:18:03
Message-ID: c8b0a862-0724-4687-950c-d584267875fc () oracle ! com
[Download RAW message or body]

Looks good. I've been using it for a while now with no issues.

Chris

On 12/20/18 4:52 AM, Gary Adams wrote:
> This should be a trivial update.
>
> The default "excludes" filter for jdb
> should have been updated when the jdk.* packages were first introduced.
>
>
> diff --git 
> a/src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/Env.java 
> b/src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/Env.java
> --- a/src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/Env.java
> +++ b/src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/Env.java
> @@ -1,5 +1,5 @@
>   /*
> - * Copyright (c) 1998, 2011, Oracle and/or its affiliates. All rights 
> reserved.
> + * Copyright (c) 1998, 2018, 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
> @@ -108,7 +108,7 @@
>
>          static private List<String> excludes() {
>                  if (excludes == null) {
> -                       setExcludes("java.*, javax.*, sun.*, com.sun.*");
> +                       setExcludes("java.*, javax.*, sun.*, com.sun.*, jdk.*");
>                  }
>                  return excludes;
>          }
>


[prev in list] [next in list] [prev in thread] [next in thread]