https://bz.apache.org/bugzilla/show_bug.cgi?id=62090

--- Comment #10 from Christopher Schultz <[email protected]> ---
(In reply to Slava Risenberg from comment #8)
> (In reply to Remy Maucherat from comment #6)
> > I'd rather add an IAE to ContainerBase.setName.
> 
> ContainerBase is an abstract class, making change in it will affect other
> irrelevant Classes like StandardContext, StandardEngine, StandardHost and
> StandardWrapper, so it's definitely not a good place for the fix.

Also, the default value for "name" is null, and this check can only be made it
setName(String) is actually called. Assuming Tomcat uses digester to parse XML
files, a missing servlet-name will skip the call to
ContainerBase.setName(String), no?

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to