Why did you discard my suggestion about PropertyTokenizer? Is there
something wrong with that? I think it would be a more general solution.

On 15.02.2006 14:07:59 adelmelle wrote:
> Author: adelmelle
> Date: Wed Feb 15 05:07:57 2006
> New Revision: 378003
> 
> URL: http://svn.apache.org/viewcvs?rev=378003&view=rev
> Log:
> Added default empty String to the border shorthand. Avoids NPE mentioned in 
> Bugzilla 38648. Still need to add a warning, since property now silently 
> falls back to the default(s).
> 
> Modified:
>     xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java
> 
> Modified: 
> xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java
> URL: 
> http://svn.apache.org/viewcvs/xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java?rev=378003&r1=378002&r2=378003&view=diff
> ==============================================================================
> --- xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java 
> (original)
> +++ xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java 
> Wed Feb 15 05:07:57 2006
> @@ -2623,6 +2623,9 @@
>          // background-position
>          m  = new ListProperty.Maker(PR_BACKGROUND_POSITION);
>          m.setInherited(false);
> +        m.addKeyword("left", "0% 50%");
> +        m.addKeyword("center", "50% 50%");
> +        m.addKeyword("right", "100% 50%");
>          m.setDefault("0% 0%");
>          m.setDatatypeParser(new BackgroundPositionShorthandParser());
>          addPropertyMaker("background-position", m);
> @@ -2630,6 +2633,7 @@
>          // border
>          m  = new ListProperty.Maker(PR_BORDER);
>          m.setInherited(false);
> +        m.setDefault("");
>          m.setDatatypeParser(new GenericShorthandParser());
>          addPropertyMaker("border", m);



Jeremias Maerki

Reply via email to