jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Felix Schumacher <felix.schumac...@internetallee.de>
Subject Re: svn commit: r1623759 - /jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeNode.java
Date Wed, 10 Sep 2014 05:39:19 GMT


On 9. September 2014 13:25:39 MESZ, pmouawad@apache.org wrote:
>Author: pmouawad
>Date: Tue Sep  9 11:25:38 2014
>New Revision: 1623759
>
>URL: http://svn.apache.org/r1623759
>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
>Put back final.
>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=1623759&r1=1623758&r2=1623759&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
>Tue Sep  9 11:25:38 2014
>@@ -47,7 +47,7 @@ public class JMeterTreeNode extends Defa
>     private static final int TEST_PLAN_LEVEL = 1;
> 
>     // See Bug 54648
>-    private transient JMeterTreeModel treeModel;
>+    private final transient JMeterTreeModel treeModel;
I don't believe that fields can be transient and final at the same time. 

Since transient is for serialization and final fields have to be set during construction.
But deserialization will not call the constructor. 

Regards
Felix
> 
>     private boolean markedBySearch;
> 


Mime
View raw message