You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/06/26 14:17:33 UTC
svn commit: r958214 - in /tomcat/trunk/webapps/examples/WEB-INF/classes:
CookieExample.java RequestHeaderExample.java RequestInfoExample.java
RequestParamExample.java SessionExample.java chat/ChatServlet.java
Author: markt
Date: Sat Jun 26 12:17:33 2010
New Revision: 958214
URL: http://svn.apache.org/viewvc?rev=958214&view=rev
Log:
Fix some Eclispe / Findbugs warnings
Modified:
tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java
tomcat/trunk/webapps/examples/WEB-INF/classes/RequestHeaderExample.java
tomcat/trunk/webapps/examples/WEB-INF/classes/RequestInfoExample.java
tomcat/trunk/webapps/examples/WEB-INF/classes/RequestParamExample.java
tomcat/trunk/webapps/examples/WEB-INF/classes/SessionExample.java
tomcat/trunk/webapps/examples/WEB-INF/classes/chat/ChatServlet.java
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java Sat Jun 26 12:17:33 2010
@@ -35,7 +35,7 @@ public class CookieExample extends HttpS
private static final long serialVersionUID = 1L;
- ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
+ private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
@Override
public void doGet(HttpServletRequest request,
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/RequestHeaderExample.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/RequestHeaderExample.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/RequestHeaderExample.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/RequestHeaderExample.java Sat Jun 26 12:17:33 2010
@@ -35,7 +35,7 @@ public class RequestHeaderExample extend
private static final long serialVersionUID = 1L;
- ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
+ private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
@Override
public void doGet(HttpServletRequest request,
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/RequestInfoExample.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/RequestInfoExample.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/RequestInfoExample.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/RequestInfoExample.java Sat Jun 26 12:17:33 2010
@@ -35,7 +35,7 @@ public class RequestInfoExample extends
private static final long serialVersionUID = 1L;
- ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
+ private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
@Override
public void doGet(HttpServletRequest request,
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/RequestParamExample.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/RequestParamExample.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/RequestParamExample.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/RequestParamExample.java Sat Jun 26 12:17:33 2010
@@ -35,7 +35,7 @@ public class RequestParamExample extends
private static final long serialVersionUID = 1L;
- ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
+ private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
@Override
public void doGet(HttpServletRequest request,
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/SessionExample.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/SessionExample.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/SessionExample.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/SessionExample.java Sat Jun 26 12:17:33 2010
@@ -35,7 +35,7 @@ public class SessionExample extends Http
private static final long serialVersionUID = 1L;
- ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
+ private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
@Override
public void doGet(HttpServletRequest request,
Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/chat/ChatServlet.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/chat/ChatServlet.java?rev=958214&r1=958213&r2=958214&view=diff
==============================================================================
--- tomcat/trunk/webapps/examples/WEB-INF/classes/chat/ChatServlet.java (original)
+++ tomcat/trunk/webapps/examples/WEB-INF/classes/chat/ChatServlet.java Sat Jun 26 12:17:33 2010
@@ -45,7 +45,7 @@ public class ChatServlet
protected ArrayList<HttpServletResponse> connections =
new ArrayList<HttpServletResponse>();
- protected MessageSender messageSender = null;
+ protected transient MessageSender messageSender = null;
@Override
public void init() throws ServletException {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
Re: svn commit: r958214 - in /tomcat/trunk/webapps/examples/WEB-INF/classes:
CookieExample.java RequestHeaderExample.java RequestInfoExample.java RequestParamExample.java
SessionExample.java chat/ChatServlet.java
Posted by Mark Thomas <ma...@apache.org>.
On 26/06/2010 14:18, sebb wrote:
> On 26/06/2010, Felix Schumacher<fe...@internetallee.de> wrote:
>> Hi Mark,
>>
>> Am Samstag, den 26.06.2010, 12:17 +0000 schrieb markt@apache.org:
>>
>>> Author: markt
>> > Date: Sat Jun 26 12:17:33 2010
>> > New Revision: 958214
>> >
>> > URL: http://svn.apache.org/viewvc?rev=958214&view=rev
>> > Log:
>> > Fix some Eclispe / Findbugs warnings
>> ...
>> > Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java
>>
>>> URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java?rev=958214&r1=958213&r2=958214&view=diff
>> > ==============================================================================
>> > --- tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java (original)
>> > +++ tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java Sat Jun 26 12:17:33 2010
>> > @@ -35,7 +35,7 @@ public class CookieExample extends HttpS
>> >
>> > private static final long serialVersionUID = 1L;
>> >
>> > - ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
>> > + private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
>>
>> shouldn't we init rb in an init method, so that it will be reinitialized
>> after deserialization, like messageSender in ChatServlet? That way we
>> would get rid of the warnings and have a correct servlet.
>
> Or could they be "private static final" instead?
That would be less work and probably more correct.
Mark
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
Re: svn commit: r958214 - in /tomcat/trunk/webapps/examples/WEB-INF/classes:
CookieExample.java RequestHeaderExample.java RequestInfoExample.java
RequestParamExample.java SessionExample.java chat/ChatServlet.java
Posted by sebb <se...@gmail.com>.
On 26/06/2010, Felix Schumacher <fe...@internetallee.de> wrote:
> Hi Mark,
>
> Am Samstag, den 26.06.2010, 12:17 +0000 schrieb markt@apache.org:
>
> > Author: markt
> > Date: Sat Jun 26 12:17:33 2010
> > New Revision: 958214
> >
> > URL: http://svn.apache.org/viewvc?rev=958214&view=rev
> > Log:
> > Fix some Eclispe / Findbugs warnings
> ...
> > Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java
>
> > URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java?rev=958214&r1=958213&r2=958214&view=diff
> > ==============================================================================
> > --- tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java (original)
> > +++ tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java Sat Jun 26 12:17:33 2010
> > @@ -35,7 +35,7 @@ public class CookieExample extends HttpS
> >
> > private static final long serialVersionUID = 1L;
> >
> > - ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
> > + private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
>
> shouldn't we init rb in an init method, so that it will be reinitialized
> after deserialization, like messageSender in ChatServlet? That way we
> would get rid of the warnings and have a correct servlet.
Or could they be "private static final" instead?
> Regards,
>
> Felix
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
Re: svn commit: r958214 - in
/tomcat/trunk/webapps/examples/WEB-INF/classes: CookieExample.java
RequestHeaderExample.java RequestInfoExample.java RequestParamExample.java
SessionExample.java chat/ChatServlet.java
Posted by Felix Schumacher <fe...@internetallee.de>.
Hi Mark,
Am Samstag, den 26.06.2010, 12:17 +0000 schrieb markt@apache.org:
> Author: markt
> Date: Sat Jun 26 12:17:33 2010
> New Revision: 958214
>
> URL: http://svn.apache.org/viewvc?rev=958214&view=rev
> Log:
> Fix some Eclispe / Findbugs warnings
...
> Modified: tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java
> URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java?rev=958214&r1=958213&r2=958214&view=diff
> ==============================================================================
> --- tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java (original)
> +++ tomcat/trunk/webapps/examples/WEB-INF/classes/CookieExample.java Sat Jun 26 12:17:33 2010
> @@ -35,7 +35,7 @@ public class CookieExample extends HttpS
>
> private static final long serialVersionUID = 1L;
>
> - ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
> + private transient ResourceBundle rb = ResourceBundle.getBundle("LocalStrings");
shouldn't we init rb in an init method, so that it will be reinitialized
after deserialization, like messageSender in ChatServlet? That way we
would get rid of the warnings and have a correct servlet.
Regards,
Felix
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org