jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philippe Mouawad <philippe.moua...@gmail.com>
Subject Re: svn commit: r1624728 - /jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
Date Sat, 13 Sep 2014 12:47:28 GMT
Sebb, I will answer in thread that triggered this revert.

On Sat, Sep 13, 2014 at 2:36 PM, sebb <sebbaz@gmail.com> wrote:

> On 13 September 2014 13:10,  <pmouawad@apache.org> wrote:
> > Author: pmouawad
> > Date: Sat Sep 13 12:10:54 2014
> > New Revision: 1624728
> >
> > URL: http://svn.apache.org/r1624728
> > Log:
> > Bug 54648 - JMeter GUI on OS X crashes when using CMD+C (keyboard
> shortcut or UI menu entry) on an element from the tree
> > Remove final as in this case transient cannot be final.
>
> Please re-instate final; you misunderstood my comment.
>
> > See:
> >
> http://mail-archives.apache.org/mod_mbox/jmeter-dev/201409.mbox/%3CCAOGo0VaKGcTm=3+jF-mrjaVFBi21HPWE7mW1u09ve-QsP6sLOA@mail.gmail.com%3E
> > Bugzilla Id: 54648
> >
> > Modified:
> >     jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
> >
> > Modified:
> jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
> > URL:
> http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java?rev=1624728&r1=1624727&r2=1624728&view=diff
> >
> ==============================================================================
> > --- jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
> (original)
> > +++ jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
> Sat Sep 13 12:10:54 2014
> > @@ -47,7 +47,7 @@ public class JMeterTreeNode extends Defa
> >      private static final int TEST_PLAN_LEVEL = 1;
> >
> >      // See Bug 54648
> > -    private final transient JMeterTreeModel treeModel;
> > +    private transient JMeterTreeModel treeModel;
> >
> >      private boolean markedBySearch;
> >
> >
> >
>



-- 
Cordialement.
Philippe Mouawad.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message