svn commit: r654190 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java

[adelmelle]

Author: adelmelle
Date: Wed May  7 10:41:07 2008
New Revision: 654190

URL: http://svn.apache.org/viewvc?rev=654190&view=rev
Log:
Added comment

Modified:
    xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java

Modified: xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java?rev=654190&r1=654189&r2=654190&view=diff
==============================================================================
--- xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java 
(original)
+++ xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/flow/Wrapper.java Wed May  
7 10:41:07 2008
@@ -67,8 +67,13 @@
                         "(#PCDATA|%inline;|%block;)");
                 }
             } else if (isBlockOrInlineItem(nsURI, localName)) {
+                /* delegate validation to parent, but keep the error reporting
+                 * tidy. If we would simply call validateChildNode() on the
+                 * parent, the user would get a wrong impression, as only the
+                 * locator (if any) will contain a reference to the offending
+                 * fo:wrapper.
+                 */
                 try {
-                    //delegate validation to parent
                     FONode.validateChildNode(this.parent, loc, nsURI, 
localName);
                 } catch (ValidationException vex) {
                     invalidChildError(loc, getName(), FO_URI, localName, 
"rule.wrapperInvalidChildForParent");



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]