You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by rw...@apache.org on 2019/06/05 18:00:47 UTC
svn commit: r1860674 - in /pivot/trunk/wtk/src/org/apache/pivot/wtk:
DesktopApplicationContext.java GraphicsUtilities.java ListView.java
Author: rwhitcomb
Date: Wed Jun 5 18:00:47 2019
New Revision: 1860674
URL: http://svn.apache.org/viewvc?rev=1860674&view=rev
Log:
PIVOT-1032: Fix the EmptyBlock and NestedBlock style errors in
"core" (that is, non-demo or tutorial) classes.
Modified:
pivot/trunk/wtk/src/org/apache/pivot/wtk/DesktopApplicationContext.java
pivot/trunk/wtk/src/org/apache/pivot/wtk/GraphicsUtilities.java
pivot/trunk/wtk/src/org/apache/pivot/wtk/ListView.java
Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/DesktopApplicationContext.java
URL: http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/DesktopApplicationContext.java?rev=1860674&r1=1860673&r2=1860674&view=diff
==============================================================================
--- pivot/trunk/wtk/src/org/apache/pivot/wtk/DesktopApplicationContext.java (original)
+++ pivot/trunk/wtk/src/org/apache/pivot/wtk/DesktopApplicationContext.java Wed Jun 5 18:00:47 2019
@@ -642,9 +642,8 @@ public final class DesktopApplicationCon
// Load the application
try {
Class<?> applicationClass = Class.forName(applicationClassName);
- if (useApplicationInstance) {
- // application has already been set, before calling this method
- } else {
+ // If the application has not already been set before calling this method...
+ if (!useApplicationInstance) {
application = (Application) applicationClass.getDeclaredConstructor().newInstance();
}
} catch (ClassNotFoundException | InstantiationException | IllegalAccessException
Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/GraphicsUtilities.java
URL: http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/GraphicsUtilities.java?rev=1860674&r1=1860673&r2=1860674&view=diff
==============================================================================
--- pivot/trunk/wtk/src/org/apache/pivot/wtk/GraphicsUtilities.java (original)
+++ pivot/trunk/wtk/src/org/apache/pivot/wtk/GraphicsUtilities.java Wed Jun 5 18:00:47 2019
@@ -342,7 +342,8 @@ public final class GraphicsUtilities {
float startX, startY;
float endX, endY;
- switch (PaintType.valueOf(paintType.toUpperCase(Locale.ENGLISH))) {
+ PaintType pType = PaintType.valueOf(paintType.toUpperCase(Locale.ENGLISH));
+ switch (pType) {
case SOLID_COLOR:
paint = decodeColor((String) JSON.get(dictionary, COLOR_KEY));
break;
@@ -357,12 +358,8 @@ public final class GraphicsUtilities {
paint = new GradientPaint(startX, startY, startColor, endX, endY, endColor);
break;
- case LINEAR_GRADIENT: {
- startX = JSON.getFloat(dictionary, START_X_KEY);
- startY = JSON.getFloat(dictionary, START_Y_KEY);
- endX = JSON.getFloat(dictionary, END_X_KEY);
- endY = JSON.getFloat(dictionary, END_Y_KEY);
-
+ case LINEAR_GRADIENT:
+ case RADIAL_GRADIENT:
@SuppressWarnings("unchecked")
List<Dictionary<String, ?>> stops = (List<Dictionary<String, ?>>) JSON.get(dictionary, STOPS_KEY);
@@ -379,37 +376,24 @@ public final class GraphicsUtilities {
colors[i] = color;
}
- paint = new LinearGradientPaint(startX, startY, endX, endY, fractions, colors);
- break;
- }
-
- case RADIAL_GRADIENT: {
- float centerX = JSON.getFloat(dictionary, CENTER_X_KEY);
- float centerY = JSON.getFloat(dictionary, CENTER_Y_KEY);
- float radius = JSON.getFloat(dictionary, RADIUS_KEY);
-
- @SuppressWarnings("unchecked")
- List<Dictionary<String, ?>> stops = (List<Dictionary<String, ?>>) JSON.get(dictionary, STOPS_KEY);
-
- int n = stops.getLength();
- float[] fractions = new float[n];
- Color[] colors = new Color[n];
- for (int i = 0; i < n; i++) {
- Dictionary<String, ?> stop = stops.get(i);
+ if (pType == PaintType.LINEAR_GRADIENT) {
+ startX = JSON.getFloat(dictionary, START_X_KEY);
+ startY = JSON.getFloat(dictionary, START_Y_KEY);
+ endX = JSON.getFloat(dictionary, END_X_KEY);
+ endY = JSON.getFloat(dictionary, END_Y_KEY);
+
+ paint = new LinearGradientPaint(startX, startY, endX, endY, fractions, colors);
+ } else {
+ float centerX = JSON.getFloat(dictionary, CENTER_X_KEY);
+ float centerY = JSON.getFloat(dictionary, CENTER_Y_KEY);
+ float radius = JSON.getFloat(dictionary, RADIUS_KEY);
- float offset = JSON.getFloat(stop, OFFSET_KEY);
- fractions[i] = offset;
-
- Color color = decodeColor((String) JSON.get(stop, COLOR_KEY));
- colors[i] = color;
+ paint = new RadialGradientPaint(centerX, centerY, radius, fractions, colors);
}
-
- paint = new RadialGradientPaint(centerX, centerY, radius, fractions, colors);
break;
- }
default:
- throw new UnsupportedOperationException();
+ throw new UnsupportedOperationException("Paint type " + paintType + " is not supported.");
}
return paint;
Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/ListView.java
URL: http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/ListView.java?rev=1860674&r1=1860673&r2=1860674&view=diff
==============================================================================
--- pivot/trunk/wtk/src/org/apache/pivot/wtk/ListView.java (original)
+++ pivot/trunk/wtk/src/org/apache/pivot/wtk/ListView.java Wed Jun 5 18:00:47 2019
@@ -1785,11 +1785,10 @@ public class ListView extends Component
item = itemsStateBindMapping.get(listData, index.intValue());
}
- if (itemsStateBindMapping == null) {
- // TODO: ?? what to do here? we need to set the MIXED state for the item
- } else {
+ if (itemsStateBindMapping != null) {
itemsStateBindMapping.setState(item, Button.State.MIXED);
}
+ // TODO: else ?? what to do here? we need to set the MIXED state for the item
items.add(item);
}
@@ -1806,11 +1805,10 @@ public class ListView extends Component
item = itemsStateBindMapping.get(listData, index.intValue());
}
- if (itemsStateBindMapping == null) {
- // TODO: ?? what to do here? we need to set the SELECTED state for the item
- } else {
+ if (itemsStateBindMapping != null) {
itemsStateBindMapping.setState(item, Button.State.SELECTED);
}
+ // TODO: else ?? what to do here? we need to set the SELECTED state for the item
items.add(item);
}