You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2013/08/03 14:48:04 UTC

svn commit: r1509977 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java

Author: sebb
Date: Sat Aug  3 12:48:04 2013
New Revision: 1509977

URL: http://svn.apache.org/r1509977
Log:
Avoid warning from serialisation tests

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java?rev=1509977&r1=1509976&r2=1509977&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java Sat Aug  3 12:48:04 2013
@@ -101,7 +101,8 @@ public class KerberosManager implements 
         return null;
     }
 
-    static private class LoginCallbackHandler implements CallbackHandler {
+    // Needs to be package-protected to avoid problem with serialisation tests
+    static class LoginCallbackHandler implements CallbackHandler {
         private final String password;
         private final String username;
 



Re: svn commit: r1509977 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java

Posted by Philippe Mouawad <ph...@gmail.com>.
Hello sebb,
Just making the KerberosManager class public fixes the warning.

Making this only package protected does not work for me.

Regards
Philippe

On Sat, Aug 3, 2013 at 2:48 PM, <se...@apache.org> wrote:

> Author: sebb
> Date: Sat Aug  3 12:48:04 2013
> New Revision: 1509977
>
> URL: http://svn.apache.org/r1509977
> Log:
> Avoid warning from serialisation tests
>
> Modified:
>
> jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java
>
> Modified:
> jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java
> URL:
> http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java?rev=1509977&r1=1509976&r2=1509977&view=diff
>
> ==============================================================================
> ---
> jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java
> (original)
> +++
> jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/KerberosManager.java
> Sat Aug  3 12:48:04 2013
> @@ -101,7 +101,8 @@ public class KerberosManager implements
>          return null;
>      }
>
> -    static private class LoginCallbackHandler implements CallbackHandler {
> +    // Needs to be package-protected to avoid problem with serialisation
> tests
> +    static class LoginCallbackHandler implements CallbackHandler {
>          private final String password;
>          private final String username;
>
>
>
>


-- 
Cordialement.
Philippe Mouawad.