Skip to content

Commit e593d43

Browse files
committed
cleanup some comments and add missing overrides
1 parent 7c0aacd commit e593d43

File tree

3 files changed

+3
-7
lines changed

3 files changed

+3
-7
lines changed

src/main/java/org/jruby/rack/DefaultRackConfig.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -166,12 +166,6 @@ protected RackLogger createLogger(final String loggerClass) {
166166
throw new RackException("could not create logger: '" + loggerClass + "'", e.getTargetException());
167167
}
168168
}
169-
//catch (ClassNotFoundException e) {
170-
// if ( ! isQuiet() ) {
171-
// err.println("failed creating logger: '" + loggerClass + "'");
172-
// e.printStackTrace(err);
173-
// }
174-
//}
175169
catch (Exception e) {
176170
if ( ! isQuiet() ) {
177171
err.println("failed creating logger: '" + loggerClass + "'");

src/main/java/org/jruby/rack/RackContext.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@
1313
* @author nicksieger
1414
*/
1515
public interface RackContext extends RackLogger {
16-
RackConfig getConfig(); /// NOTE: deprecate ?
16+
RackConfig getConfig();
1717
String getServerInfo();
1818
}

src/main/java/org/jruby/rack/RackServletContextListener.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ public RackServletContextListener() {
3939
this.factory = factory;
4040
}
4141

42+
@Override
4243
public void contextInitialized(final ServletContextEvent event) {
4344
final ServletContext context = event.getServletContext();
4445
final ServletRackConfig config = new ServletRackConfig(context);
@@ -54,6 +55,7 @@ public void contextInitialized(final ServletContextEvent event) {
5455
}
5556
}
5657

58+
@Override
5759
public void contextDestroyed(final ServletContextEvent event) {
5860
final ServletContext context = event.getServletContext();
5961
final RackApplicationFactory factory =

0 commit comments

Comments
 (0)