Rendering refactor

src/main/java/de/mfo/jsurf/rendering/cpu/CPUAlgebraicSurfaceRendererExt.java

@@ -25,10 +25,8 @@ private DrawcallStaticDataExt( DrawcallStaticData dcsd )
         public int[] getColorBuffer() { return privateDcsd.colorBuffer; }
     }
 
-    public CPUAlgebraicSurfaceRendererExt()
-    {
-        super();
-    }
+    private Integer degree;
+    private DegreeCalculator degreeCalculator;
 
     public synchronized DrawcallStaticDataExt collectDrawCallStaticDataExt( int[] colorBuffer, int width, int height )
     {
@@ -37,58 +35,61 @@ public synchronized DrawcallStaticDataExt collectDrawCallStaticDataExt( int[] co
 
     public synchronized void draw( DrawcallStaticDataExt dcsd )
     {
-        final int width = dcsd.privateDcsd.width;
-        final int height = dcsd.privateDcsd.height;
-
-        if( width == 0 || height == 0 )
+    	if (!validArea(dcsd))
             return;
 
-        int xStep = width / Math.min( width, Math.max( 2, Runtime.getRuntime().availableProcessors() ) );
-        int yStep = height / Math.min( height, 3 );//Math.max( 2, Runtime.getRuntime().availableProcessors() ) );
-
-        boolean success = true;
-
-        LinkedList< FutureTask< Boolean > > tasks = new LinkedList< FutureTask< Boolean > >();
-        for( int x = 0; x < width; x += xStep )
-            for( int y = 0; y < height; y += yStep )
-                tasks.add( new FutureTask< Boolean >( new RenderingTask( dcsd.privateDcsd, x, y, Math.min( x + xStep, width - 1 ), Math.min( y + yStep, height - 1 ) ) ) );
+    	List<FutureTask<Boolean>> tasks = createRenderTasks(dcsd);
 
-        renderingTasks = tasks;
+    	scheduleAndWait(tasks);
+    }
+
+    private void scheduleAndWait(List<FutureTask<Boolean>> tasks) {
+        boolean success = false;
+        try {
+        	tasks.forEach( threadPoolExecutor::execute );
 
-        try
-        {
-            for( FutureTask< Boolean > task : tasks )
-                threadPoolExecutor.execute( task );
             for( FutureTask< Boolean > task : tasks )
-                success = success && task.get();
-        }
-        catch( ExecutionException ie )
-        {
-            success = false;
-        }
-        catch( InterruptedException ie )
-        {
-            super.stopDrawing();
-            success = false;
-        }
-        catch( RejectedExecutionException ree )
-        {
-            success = false;
-        }
-        catch( CancellationException ree )
-        {
-            success = false;
-        }
-        finally
-        {
+                if (!task.get())
+                	break;
+
+            success = true;
+        } catch( ExecutionException | RejectedExecutionException | CancellationException e ) {
+        } catch( InterruptedException ie ) {
+        	stopTasks(tasks);
+        } finally {
             if( !success || Thread.interrupted() )
                 throw new RenderingInterruptedException( "Rendering interrupted" );
         }
     }
+
+	// TODO: the parent method stopDrawing uses implicitly the renderingTasks field, but it's not actually needed
+    private void stopTasks(List<FutureTask<Boolean>> tasks) {
+    	tasks.forEach( (task) -> task.cancel(true) );
+    }
+
+    private boolean validArea(DrawcallStaticDataExt callData) {
+    	return callData.privateDcsd.width != 0 && callData.privateDcsd.height != 0;
+    }
 
-    private Integer degree;
-    private DegreeCalculator degreeCalculator;
-
+	private List<FutureTask<Boolean>> createRenderTasks(DrawcallStaticDataExt dcsd) {
+		int processors = Runtime.getRuntime().availableProcessors();
+        int xStep = dcsd.privateDcsd.width / Math.min( dcsd.privateDcsd.width, Math.max( 2, processors ) );
+        int yStep = dcsd.privateDcsd.height / Math.min( dcsd.privateDcsd.height, 3 );
+
+        List< FutureTask< Boolean > > tasks = new LinkedList< FutureTask< Boolean > >();
+        for( int x = 0; x < dcsd.privateDcsd.width; x += xStep )
+            for( int y = 0; y < dcsd.privateDcsd.height; y += yStep )
+                tasks.add( createRenderTask(dcsd, x, y, xStep, yStep) );
+
+		return tasks;
+	}
+
+	FutureTask<Boolean> createRenderTask(DrawcallStaticDataExt dcsd, int x, int y, int xStep, int yStep) {
+		int xEnd = Math.min( x + xStep, dcsd.privateDcsd.width - 1 );
+		int yEnd = Math.min( y + yStep, dcsd.privateDcsd.height - 1 );
+        return new FutureTask< Boolean >(new RenderingTask( dcsd.privateDcsd, x, y, xEnd, yEnd ) );
+	}
+
     @Override
     public void setSurfaceFamily( String expression )
         throws Exception

src/main/java/de/mfo/surfer/Main.java

@@ -46,7 +46,7 @@ public class Main extends Application
     {
         try
         {
-            fxmlRoot = ( Group ) new FXMLLoader().load(
+            fxmlRoot = ( Group ) FXMLLoader.load(
                 getClass().getResource( "/de/mfo/surfer/fxml/surfer_touchscreen_1920_x_1080.fxml" )
             );
         }

src/main/java/de/mfo/surfer/control/ColorPickerPanel.java

@@ -2,7 +2,6 @@
 
 import javafx.beans.property.ObjectProperty;
 import javafx.geometry.Pos;
-import javafx.scene.control.ColorPicker;
 import javafx.scene.layout.Priority;
 import javafx.scene.layout.Region;
 import javafx.scene.layout.VBox;

src/main/java/de/mfo/surfer/control/LanguageSelector.java

@@ -8,7 +8,6 @@
 
 import javafx.beans.binding.Bindings;
 import javafx.collections.ObservableList;
-import javafx.geometry.Bounds;
 import javafx.scene.control.ChoiceBox;
 import javafx.util.StringConverter;
 

src/main/java/de/mfo/surfer/control/MiscSceneNodeButtonPanel.java

@@ -3,7 +3,6 @@
 import de.mfo.surfer.control.SceneNodeButton;
 import de.mfo.surfer.Main;
 
-import java.awt.*;
 import java.io.File;
 import java.util.EnumMap;
 import java.util.function.Consumer;
@@ -12,7 +11,6 @@
 import javafx.beans.property.BooleanProperty;
 import javafx.event.ActionEvent;
 import javafx.event.EventHandler;
-import javafx.print.PrinterJob;
 import javafx.scene.Group;
 import javafx.scene.Node;
 import javafx.scene.layout.Region;

src/main/java/de/mfo/surfer/control/PreferenceDialog.java

@@ -122,7 +122,7 @@ else if( ObjectProperty.class.isAssignableFrom( p.getClass() ) )
         {
             Class<?> c = Utils.wrapInRte( () -> (Class<?>) m.getDeclaringClass().getMethod(m.getName().replaceAll("Property$", "Type")).invoke( null));
             if( File.class.isAssignableFrom( c ) ) {
-                StringConverter stringConverter = new StringConverter<File>() {
+                StringConverter<File> stringConverter = new StringConverter<File>() {
                     @Override
                     public String toString(File file) {
                         return file == null ? "" : file.toString();
@@ -174,7 +174,7 @@ private Node createDoubleEditor( DoubleProperty dp )
     {
         final double originalValue = dp.get();
 
-        Spinner< Double > spinner = new Spinner( Double.MIN_VALUE, Double.MAX_VALUE, originalValue, 0.1 );
+        Spinner< Double > spinner = new Spinner< Double >( Double.MIN_VALUE, Double.MAX_VALUE, originalValue, 0.1 );
         spinner.setEditable( true );
         spinner.getValueFactory().valueProperty().bindBidirectional( dp.asObject() );
         resetters.add( () -> dp.set( originalValue ) );
@@ -186,7 +186,7 @@ private Node createIntegerEditor( IntegerProperty ip )
     {
         final int originalValue = ip.get();
 
-        Spinner< Integer > spinner = new Spinner( Integer.MIN_VALUE, Integer.MAX_VALUE, originalValue, 1 );
+        Spinner< Integer > spinner = new Spinner< Integer >( Integer.MIN_VALUE, Integer.MAX_VALUE, originalValue, 1 );
         spinner.setEditable( true );
         spinner.getValueFactory().valueProperty().bindBidirectional( ip.asObject() );
         resetters.add( () -> ip.set( originalValue ) );

src/main/java/de/mfo/surfer/control/PrintDialog.java

@@ -260,8 +260,10 @@ public void changed( ObservableValue<? extends Worker.State> observable, Worker.
         else
         {
             // pass HTML content into the webEngine instead of the path to index.html because otherwise it won't resolve file:// URLs
-            String html = new Scanner(PrintDialog.class.getResourceAsStream( resourceName ), "UTF-8").useDelimiter("\\A").next();
-            webEngine.loadContent(html);
+        	try ( Scanner scanner = new Scanner(PrintDialog.class.getResourceAsStream( resourceName ), "UTF-8") ) {
+                String html = scanner.useDelimiter("\\A").next();
+                webEngine.loadContent(html);
+        	}
         }
     }