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: r1739924 - /jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java
Date Tue, 19 Apr 2016 12:54:22 GMT


Am 19. April 2016 14:40:57 MESZ, schrieb sebb@apache.org:
>Author: sebb
>Date: Tue Apr 19 12:40:57 2016
>New Revision: 1739924
>
>URL: http://svn.apache.org/viewvc?rev=1739924&view=rev
>Log:
>Field can still be final
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java
>URL:
>http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java?rev=1739924&r1=1739923&r2=1739924&view=diff
>==============================================================================
>---
>jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java
>(original)
>+++
>jmeter/trunk/src/core/org/apache/jmeter/logging/LogkitLoggerAdapter.java
>Tue Apr 19 12:40:57 2016
>@@ -32,10 +32,8 @@ import org.slf4j.helpers.MessageFormatte
>  */
>public class LogkitLoggerAdapter extends MarkerIgnoringBase implements
>Serializable {
> 
>-    transient Logger logger;
>-    /**
>-     * 
>-     */
>+    final transient Logger logger;

Is this really valid? When an object of This class gets deserialized, its transient fields
will not be initialized. But they are marked as non changing and thus stay in that state.


Looks strange to me. 

Felix

>+
>     private static final long serialVersionUID = -122848886791823355L;
> 
>     /**
>@@ -44,6 +42,7 @@ public class LogkitLoggerAdapter extends
>     @Deprecated // only for Unit test usage
>     public LogkitLoggerAdapter() {
>         super();
>+        this.logger = null;
>     }
>     
>     LogkitLoggerAdapter(org.apache.log.Logger logkitLogger) {


Mime
View raw message