#896 Redundant GenericServlet.destroy() invocation
Closed: Fixed None Opened 10 years ago by edewata.

The CMSStartServlet currently calls the super.destroy() method:

public void destroy() {
    CMS.shutdown();
    super.destroy();
}

This invocation is unnecessary because the super.destroy() is an empty method in GenericServlet.

http://svn.apache.org/repos/asf/tomcat/tc7.0.x/trunk/java/javax/servlet/GenericServlet.java

public void destroy() {
    // NOOP by default
}

GenericServlet is a convenience class that provides default (i.e. empty) implementation for most methods in the Servlet interface.


master: ae38b34e717b386f06407db9e5a4723472b77775

Metadata Update from @edewata:
- Issue assigned to edewata
- Issue set to the milestone: 10.2 - 03/14 (March)

7 years ago

Dogtag PKI is moving from Pagure issues to GitHub issues. This means that existing or new
issues will be reported and tracked through Dogtag PKI's GitHub Issue tracker.

This issue has been cloned to GitHub and is available here:
https://github.com/dogtagpki/pki/issues/1463

If you want to receive further updates on the issue, please navigate to the
GitHub issue and click on Subscribe button.

Thank you for understanding, and we apologize for any inconvenience.

Login to comment on this ticket.

Metadata