RE: Maverick servlet init parameter name constants should be public?

"Schnitzer, Jeff" <[email protected]>
Newsgroups gmane.comp.web.maverick.general
Message-ID <[email protected]>
Fixed in CVS.

J2FF

> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]]
> Sent: Thursday, April 24, 2003 1:56 AM
> To: [email protected]
> Subject: [Mav-user] Maverick servlet init parameter name constants
should
> be public?
> 
> Hi,
> 
> In one of my Maverick applications, the Dispatcher servlet is running
in
> Jetty
> (www.mortbay.org). It is configured programmatically, rather than by a
> web.xml
> file.
> 
> I need to set a value for the "configFile" servlet initialisation
> parameter. I
> hoped to be able to use the constant INITPARAM_CONFIG_FILE, defined in
> Dispatcher, in my call. For instance: -
> 
>   mavHolder.setInitParameter(INITPARAM_CONFIG_FILE, "foo");
> 
> But I cannot do that, because INITPARAM_CONFIG_FILE is declared
> "protected", not
> "public". So I have to: -
> 
>   mavHolder.setInitParameter("configFile", "foo");
> 
> This works fine, but is a bit less tidy, probably wastes a few bytes
of
> memory,
> and would break if you changed the parameter name (though perhaps
that's
> unlikely).
> 
> So my suggestion for a very simple improvement for a future version
would
> be to
> make this constant, and similar ones, public. This would tidy up code
for
> people
> doing programmatic configuration, like me.
> 
> #:^P
> 
> 
> 
> 
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> [INVALID FOOTER]



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
[INVALID FOOTER]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.