Review request for JMC-65577: Thread lanes on Threads page are not persisted

Henrik Dafgård hdafgard at gmail.com
Wed Oct 16 11:58:26 UTC 2019


Looks good to me!


Cheers,
Henrik Dafgård

On Wed, 16 Oct 2019 at 13:53, Klara Ward <klara.ward at gmail.com> wrote:

> Please review my fix for JMC 7.1.0
>
> Bug: https://bugs.openjdk.java.net/browse/JMC-6557
>
> Since this is oneline fix, just including the diff below
>
> // klaraw
>
> ===================================
> diff --git
> a/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/ThreadsPage.java
>
> b/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/ThreadsPage.java
>
>
> ---
>
> a/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/ThreadsPage.java
> +++
>
> b/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/ThreadsPage.java
> @@ -326,6 +326,7 @@
>                  public void saveTo(IWritableState state) {
>                          super.saveTo(state);
> tableFilterComponent.saveState(state.createChild(THREADS_TABLE_FILTER));
> +                       lanes.saveTo(state);
>                          saveToLocal();
>                  }
>
> --
// Henrik Dafgård


More information about the jmc-dev mailing list