Author: vhennebert
Date: Fri Jul 11 17:41:49 2014
New Revision: 1609754

URL: http://svn.apache.org/r1609754
Log:
Simplified makeShading and makePattern methods

Modified:
    
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/GradientFactory.java
    
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PDFGradientFactory.java
    
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PSGradientFactory.java

Modified: 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/GradientFactory.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/GradientFactory.java?rev=1609754&r1=1609753&r2=1609754&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/GradientFactory.java
 (original)
+++ 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/GradientFactory.java
 Fri Jul 11 17:41:49 2014
@@ -77,12 +77,12 @@ public abstract class GradientFactory<P 
         List<Double> matrix = createTransform(gradient, baseTransform, 
transform);
         List<Double> bounds = createBounds(gradient);
         List<Function> functions = createFunctions(gradient);
-        //Gradients are currently restricted to sRGB
-        PDFDeviceColorSpace colSpace = new 
PDFDeviceColorSpace(PDFDeviceColorSpace.DEVICE_RGB);
+        // Gradients are currently restricted to sRGB
+        PDFDeviceColorSpace colorSpace = new 
PDFDeviceColorSpace(PDFDeviceColorSpace.DEVICE_RGB);
         Function function = new Function(3, null, null, functions, bounds, 
null);
-        Shading shading = makeShading(gradient instanceof LinearGradientPaint 
? 2 : 3,
-                colSpace, null, null, false, coords, null, function, null);
-        return makePattern(2, shading, null, null, matrix);
+        Shading shading = makeShading(gradient instanceof LinearGradientPaint 
? 2 : 3, colorSpace,
+                coords, function);
+        return makePattern(2, shading, matrix);
     }
 
     private List<Double> createTransform(MultipleGradientPaint gradient,
@@ -149,11 +149,8 @@ public abstract class GradientFactory<P 
         return gradientColors;
     }
 
-    public abstract Shading makeShading(int shadingType,
-            PDFDeviceColorSpace colorSpace, List<Double> background, 
List<Double> bbox,
-            boolean antiAlias, List<Double> coords, List<Double> domain,
-            Function function, List<Integer> extend);
+    protected abstract Shading makeShading(int shadingType, 
PDFDeviceColorSpace colorSpace,
+            List<Double> coords, Function function);
 
-    public abstract P makePattern(int patternType, Shading shading, List xuid,
-            StringBuffer extGState, List<Double> matrix);
+    protected abstract P makePattern(int patternType, Shading shading, 
List<Double> matrix);
 }

Modified: 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PDFGradientFactory.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PDFGradientFactory.java?rev=1609754&r1=1609753&r2=1609754&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PDFGradientFactory.java
 (original)
+++ 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PDFGradientFactory.java
 Fri Jul 11 17:41:49 2014
@@ -35,27 +35,22 @@ public class PDFGradientFactory extends 
     }
 
     @Override
-    public Shading makeShading(int shadingType,
-            PDFDeviceColorSpace colorSpace, List<Double> background, 
List<Double> bbox,
-            boolean antiAlias, List<Double> coords, List<Double> domain,
-            Function function, List<Integer> extend) {
+    protected Shading makeShading(int shadingType, PDFDeviceColorSpace 
colorSpace,
+            List<Double> coords, Function function) {
         List<PDFFunction> pdfFunctions = new 
ArrayList<PDFFunction>(function.getFunctions().size());
         for (Function f : function.getFunctions()) {
             pdfFunctions.add(graphics2D.registerFunction(new PDFFunction(f)));
         }
         PDFFunction pdfFunction = graphics2D.registerFunction(new 
PDFFunction(function, pdfFunctions));
-        PDFShading newShading = new PDFShading(shadingType, colorSpace, 
background,
-                    bbox, antiAlias, coords, domain, pdfFunction, extend);
-        newShading = graphics2D.registerShading(newShading);
-        return newShading;
+        PDFShading shading = new PDFShading(shadingType, colorSpace, null, 
null, false,
+                coords, null, pdfFunction, null);
+        return graphics2D.registerShading(shading);
     }
 
     @Override
-    public PDFPattern makePattern(int patternType, Shading shading, List xuid,
-            StringBuffer extGState, List<Double> matrix) {
-        PDFPattern newPattern = new PDFPattern(patternType, shading, xuid, 
extGState, matrix);
-        newPattern = graphics2D.registerPattern(newPattern);
-        return newPattern;
+    protected PDFPattern makePattern(int patternType, Shading shading, 
List<Double> matrix) {
+        PDFPattern pattern = new PDFPattern(patternType, shading, null, null, 
matrix);
+        return graphics2D.registerPattern(pattern);
     }
 
 }

Modified: 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PSGradientFactory.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PSGradientFactory.java?rev=1609754&r1=1609753&r2=1609754&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PSGradientFactory.java
 (original)
+++ 
xmlgraphics/fop/branches/FOP-2393_gradient-rendering/src/java/org/apache/fop/render/shading/PSGradientFactory.java
 Fri Jul 11 17:41:49 2014
@@ -26,18 +26,13 @@ import org.apache.fop.render.ps.svg.PSSh
 public class PSGradientFactory extends GradientFactory<PSPattern> {
 
     @Override
-    public Shading makeShading(int shadingType,
-            PDFDeviceColorSpace colorSpace, List<Double> background, 
List<Double> bbox,
-            boolean antiAlias, List<Double> coords, List<Double> domain,
-            Function function, List<Integer> extend) {
-        Shading newShading = new PSShading(shadingType, colorSpace, 
background, bbox,
-                    antiAlias, coords, domain, function, extend);
-        return newShading;
+    protected Shading makeShading(int shadingType,
+            PDFDeviceColorSpace colorSpace, List<Double> coords, Function 
function) {
+        return new PSShading(shadingType, colorSpace, null, null, false, 
coords, null, function, null);
     }
 
     @Override
-    public PSPattern makePattern(int patternType, Shading shading, List xuid,
-            StringBuffer extGState, List<Double> matrix) {
-        return new PSPattern(patternType, shading, xuid, extGState, matrix);
+    protected PSPattern makePattern(int patternType, Shading shading, 
List<Double> matrix) {
+        return new PSPattern(patternType, shading, null, null, matrix);
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to