You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-commits@xmlgraphics.apache.org by ad...@apache.org on 2008/05/07 19:41:09 UTC

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

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: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org