You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by "Jean-Louis MONTEIRO (JIRA)" <ji...@apache.org> on 2014/05/23 17:35:02 UTC

[jira] [Commented] (TOMEE-1221) Unused code segment on tomee-catalina TomcatWebAppBuilder.java

    [ https://issues.apache.org/jira/browse/TOMEE-1221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14007247#comment-14007247 ] 

Jean-Louis MONTEIRO commented on TOMEE-1221:
--------------------------------------------

Added you in the contributor group
Assigned that issue to you
Reviewed and committed for you. Thanks

$ svn ci -m "TOMEE-1221 Unused code segment on tomee-catalina TomcatWebAppBuilder.java. Thanks Chamil Jeewantha for the patch" tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/TomcatWebAppBuilder.java
Sending        tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/TomcatWebAppBuilder.java
Transmitting file data .
Committed revision 1597117.


> Unused code segment on tomee-catalina TomcatWebAppBuilder.java
> --------------------------------------------------------------
>
>                 Key: TOMEE-1221
>                 URL: https://issues.apache.org/jira/browse/TOMEE-1221
>             Project: TomEE
>          Issue Type: Improvement
>    Affects Versions: 1.7.0, 1.6.0.1, 1.6.0.2
>         Environment: N/A
>            Reporter: Chamil Jeewantha
>            Assignee: Chamil Jeewantha
>            Priority: Trivial
>             Fix For: 1.7.0
>
>         Attachments: TOMEE-1221-patch.diff
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> The following code segment is unused hence should be removed.
>         if (standardContext.getConfigFile() == null) {
>             final String s = File.pathSeparator;
>             final File contextXmlFile = new File(standardContext.getDocBase() + s + "META-INF" + s + "context.xml");
>             if (contextXmlFile.exists()) {
>                 BackportUtil.getAPI().setConfigFile(standardContext, contextXmlFile);
>                 standardContext.setOverride(true);
>             }
>         }
> Relevant Mail Thread Subject:  Why File.pathSeparator used to create the context.xml path - msg#41759



--
This message was sent by Atlassian JIRA
(v6.2#6252)