Author: nbubna
Date: Fri Oct 24 16:28:06 2008
New Revision: 707788

URL: http://svn.apache.org/viewvc?rev=707788&view=rev
Log:
pay more attention to the comments being copied/pasted

Modified:
    
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
    
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java

Modified: 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
URL: 
http://svn.apache.org/viewvc/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java?rev=707788&r1=707787&r2=707788&view=diff
==============================================================================
--- 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
 (original)
+++ 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
 Fri Oct 24 16:28:06 2008
@@ -114,10 +114,6 @@
     @Override
     protected void configure(ValueParser props)
     {
-        // request values override configured defaults 
-        //NOTE: not sure this is the most intuitive way in all cases;
-        // it might make sense to provide the option of whether req/res
-        // values override configured ones or vice versa.
         super.configure(props);
 
         this.application = 
(ServletContext)props.getValue(ViewContext.SERVLET_CONTEXT_KEY);

Modified: 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java
URL: 
http://svn.apache.org/viewvc/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java?rev=707788&r1=707787&r2=707788&view=diff
==============================================================================
--- 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java
 (original)
+++ 
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java
 Fri Oct 24 16:28:06 2008
@@ -62,10 +62,6 @@
     @Override
     protected void configure(ValueParser props)
     {
-        // request values override configured defaults 
-        //NOTE: not sure this is the most intuitive way in all cases;
-        // it might make sense to provide the option of whether req/res
-        // values override configured ones or vice versa.
         super.configure(props);
 
         this.application = 
(ServletContext)props.getValue(ViewContext.SERVLET_CONTEXT_KEY);


Reply via email to