Changeset 1159 for trunk/src/org/expeditee/items/widgets

Timestamp:

09/21/18 13:45:34 (6 years ago)

Author:

bln4

Message:

 

File:

• trunk/src/org/expeditee/items/widgets/Widget.java (modified) (69 diffs)

trunk/src/org/expeditee/items/widgets/Widget.java

@@ -148 +148 @@
          * "org.expeditee.items."
          */
-        public static Widget createWidget(Text source) throws InteractiveWidgetNotAvailableException,
+        public static Widget createWidget(final Text source) throws InteractiveWidgetNotAvailableException,
                                                                                                                                         InteractiveWidgetInitialisationFailedException
         {
 
-                if (source == null) throw new NullPointerException("source");
-                
-                if (source.getParent() == null) throw new IllegalArgumentException("source's parent is null, InteractiveWidget's must be created from Text items with non-null parents");
-
-                String TAG = ItemUtils.GetTag(ItemUtils.TAG_IWIDGET);
+                if (source == null) {
+                        throw new NullPointerException("source");
+                }
+                
+                if (source.getParent() == null) {
+                        throw new IllegalArgumentException("source's parent is null, InteractiveWidget's must be created from Text items with non-null parents");
+                }
+
+                final String TAG = ItemUtils.GetTag(ItemUtils.TAG_IWIDGET);
 
                 String text = source.getText();
@@ -166 +170 @@
 
                 // Check starts with the widget tag and separator
-                if (!text.startsWith(TAG + ":")) throw new IllegalArgumentException("Source text must begin with \"" + TAG + ":\"");
+                if (!text.startsWith(TAG + ":")) {
+                        throw new IllegalArgumentException("Source text must begin with \"" + TAG + ":\"");
+                }
 
                 // skip over the '@iw:' preamble
@@ -191 +197 @@
                 //
                 
-                String tokens_str = (index == -1) ? text : text.substring(0, index);
+                final String tokens_str = (index == -1) ? text : text.substring(0, index);
 
                 String[] tokens = Text.parseArgsApache(tokens_str);
@@ -203 +209 @@
                 
                 // create the command line parser
-                CommandLineParser parser = new GnuParser();
+                final CommandLineParser parser = new GnuParser();
 
                 // create the Options
-                Options options = new Options();
+                final Options options = new Options();
                 options.addOption( "al", "anchorleft",   true, "Anchor the vertical left-hand edge of the interactive widget to the value provided " );
                 options.addOption( "ar", "anchorright",  true, "Anchor the vertical right-hand edge of the interactive widget to the value provided " );
@@ -259 +265 @@
                     
                 }
-                catch( ParseException exp ) {
+                catch( final ParseException exp ) {
                     System.err.println( "Unexpected exception:" + exp.getMessage() );
                     core_line = null;
@@ -265 +271 @@
                 }
                 
-                HelpFormatter formatter = new HelpFormatter();
-                StringWriter usage_str_writer = new StringWriter();
-            PrintWriter printWriter = new PrintWriter(usage_str_writer);
+                final HelpFormatter formatter = new HelpFormatter();
+                final StringWriter usage_str_writer = new StringWriter();
+            final PrintWriter printWriter = new PrintWriter(usage_str_writer);
             
             String widget_name = tokens[0];
@@ -296 +302 @@
                         }
 
-                } catch (NumberFormatException nfe) {
+                } catch (final NumberFormatException nfe) {
                         throw new IllegalArgumentException(
                                         "Bad width or height given in source text", nfe);
                 }
 
-                if (tokens.length > 3)
+                if (tokens.length > 3) {
                         throw new IllegalArgumentException(
                                         "to many arguments given before \":\" in source text");
+                }
 
                 String classname = tokens[0];
@@ -311 +318 @@
                 
                 // Attempt to locate the class using reflection
-                Class<?> iwclass = findIWidgetClass(classname);
-
-                if (iwclass == null) // ensure it exists
+                final Class<?> iwclass = findIWidgetClass(classname);
+
+                if (iwclass == null) {
                         throw new InteractiveWidgetNotAvailableException(classname
                                         + " does not exist or is not an InteractiveWidget");
+                }
 
                 
@@ -329 +337 @@
                 if (index > 0) { // index of the first ":"
                         if (text.length()>(index+1)) {
-                                String args_str = text.substring(index + 1);
+                                final String args_str = text.substring(index + 1);
                                 
                                  args = Text.parseArgsApache(args_str);
@@ -339 +347 @@
                 try {
                         // Instantiate the widget - passing the params
-                        Class parameterTypes[] = new Class[] { Text.class, String[].class };
-                        Constructor ct = iwclass.getConstructor(parameterTypes);
-
-                        Object arglist[] = new Object[] { source, args };
+                        final Class parameterTypes[] = new Class[] { Text.class, String[].class };
+                        final Constructor ct = iwclass.getConstructor(parameterTypes);
+
+                        final Object arglist[] = new Object[] { source, args };
 
                         inst = (Widget) ct.newInstance(arglist);
-                } catch (Exception e) {
+                } catch (final Exception e) {
                         throw new InteractiveWidgetNotAvailableException(
                                         "Failed to create instance via reflection: " + e.toString(),
@@ -352 +360 @@
                 
                 // Check that the widget is suitable for the current platform
-                Ecosystem systemType = EcosystemManager.getType();
+                final Ecosystem systemType = EcosystemManager.getType();
                 if (!inst.isSupportedOnEcosystem(systemType)) {
                         // Try create a JavaFXSwingWidget if creating a Swing widget in JavaFX
@@ -383 +391 @@
                 
             if(core_line.hasOption( "anchorleft" ) ) {
-                String al_str = core_line.getOptionValue( "anchorleft" );
+                final String al_str = core_line.getOptionValue( "anchorleft" );
                 
                 anchor_left = (int) Float.parseFloat(al_str);
@@ -389 +397 @@
             
             if(core_line.hasOption( "anchorright" ) ) {
-                String ar_str = core_line.getOptionValue( "anchorright" );
+                final String ar_str = core_line.getOptionValue( "anchorright" );
                 
                 anchor_right = (int) Float.parseFloat(ar_str);
@@ -395 +403 @@
             
             if(core_line.hasOption( "anchortop" ) ) {
-                String at_str = core_line.getOptionValue( "anchortop" );
+                final String at_str = core_line.getOptionValue( "anchortop" );
                 
                 anchor_top = (int) Float.parseFloat(at_str);
@@ -401 +409 @@
             
             if(core_line.hasOption( "anchorbottom" ) ) {
-                String ab_str = core_line.getOptionValue( "anchorbottom" );
+                final String ab_str = core_line.getOptionValue( "anchorbottom" );
                 
                 anchor_bottom = (int) Float.parseFloat(ab_str);
@@ -420 +428 @@
         private static Class findIWidgetClass(String classname)
         {
-                if(classname == null)
+                if(classname == null) {
                         return null;
+                }
                 // try just the classname
                 try {
-                        Class c = Class.forName(classname); // attempt to find the class
+                        final Class c = Class.forName(classname); // attempt to find the class
 
                         // If one is found, ensure that it is a descendant of an
@@ -431 +440 @@
                                         && superclass != Item.class; superclass = superclass
                                         .getSuperclass()) {
-                                if (superclass == Widget.class)
+                                if (superclass == Widget.class) {
                                         return c;
-                        }
-
-                } catch (ClassNotFoundException e) {
+                                }
+                        }
+
+                } catch (final ClassNotFoundException e) {
                 }
                 // see if the class is a widget with invalid capitalisation, or missing the widget package prefix
@@ -442 +452 @@
                 }
                 try {
-                        Class c = Class.forName(Actions.getClassName(classname)); // attempt to find the class
+                        final Class c = Class.forName(Actions.getClassName(classname)); // attempt to find the class
 
                         // If one is found, ensure that it is a descendant of an
@@ -449 +459 @@
                                         && superclass != Item.class; superclass = superclass
                                         .getSuperclass()) {
-                                if (superclass == Widget.class)
+                                if (superclass == Widget.class) {
                                         return c;
-                        }
-
-                } catch (ClassNotFoundException e) {
+                                }
+                        }
+
+                } catch (final ClassNotFoundException e) {
                 }
 
@@ -522 +533 @@
          * 
          */
-        protected Widget(Text source, int minWidth, int maxWidth, int minHeight, int maxHeight)
-        {
-                if (source == null) throw new NullPointerException("source");
+        protected Widget(final Text source, final int minWidth, final int maxWidth, final int minHeight, final int maxHeight)
+        {
+                if (source == null) {
+                        throw new NullPointerException("source");
+                }
                 if (source.getParent() == null) {
                         throw new IllegalArgumentException("source's parent is null, InteractiveWidget's must be created from Text items with non-null parents");
@@ -533 +546 @@
                 setSizeRestrictions(minWidth, maxWidth, minHeight, maxHeight); // throws IllegalArgumentException's
 
-                int x = source.getX();
-                int y = source.getY();
-                int width = (int) ((_minWidth < 0) ? 10 : _minWidth);
-                int height = (int) ((_minHeight < 0) ? 10 : _minHeight);
-
-                Frame idAllocator = _textRepresentation.getParent();
+                final int x = source.getX();
+                final int y = source.getY();
+                final int width = (_minWidth < 0) ? 10 : _minWidth;
+                final int height = (_minHeight < 0) ? 10 : _minHeight;
+
+                final Frame idAllocator = _textRepresentation.getParent();
 
                 // create WidgetCorners
@@ -552 +565 @@
                 _l4 = new WidgetEdge(_d4, _d1, idAllocator.getNextItemID(), this);
 
-                Collection<Item> enclist = new ArrayList<Item>(4);
+                final Collection<Item> enclist = new ArrayList<Item>(4);
                 enclist.add(_d1);
                 enclist.add(_d2);
@@ -601 +614 @@
          * 
          */
-        private void setSizeRestrictions(int minWidth, int maxWidth, int minHeight, int maxHeight)
+        private void setSizeRestrictions(final int minWidth, final int maxWidth, final int minHeight, final int maxHeight)
         {
                 _minWidth = minWidth;
@@ -649 +662 @@
         public Widget copy() throws InteractiveWidgetNotAvailableException, InteractiveWidgetInitialisationFailedException
         {
-                Text t = _textRepresentation.copy();
-                String clonedAnnotation = getAnnotationString();
+                final Text t = _textRepresentation.copy();
+                final String clonedAnnotation = getAnnotationString();
                 t.setText(clonedAnnotation);
                 t.setData(getData());
@@ -664 +677 @@
                 Frame parent = getParentFrame();
                 
-                if (parent == null) parent = DisplayController.getCurrentFrame();
+                if (parent == null) {
+                        parent = DisplayController.getCurrentFrame();
+                }
 
                 if (parent != null) {
-                        for (Item i : _expediteeItems) {
+                        for (final Item i : _expediteeItems) {
                                 i.setID(parent.getNextItemID());
                         }
@@ -684 +699 @@
                 // Build the annotation string such that it represents this widgets
                 // current state
-                String newAnnotation = getAnnotationString();
+                final String newAnnotation = getAnnotationString();
 
                 // Set the new text
@@ -712 +727 @@
 
                 // Create tag and append classname
-                StringBuilder sb = new StringBuilder(ItemUtils.GetTag(ItemUtils.TAG_IWIDGET));
+                final StringBuilder sb = new StringBuilder(ItemUtils.GetTag(ItemUtils.TAG_IWIDGET));
                 sb.append(':');
                 sb.append(' ');
@@ -741 +756 @@
 
                 // Append arguments if any
-                String stateArgs = Util.formatArgs(getArgs());
+                final String stateArgs = Util.formatArgs(getArgs());
                 if (stateArgs != null) {
                         sb.append(':');
@@ -784 +799 @@
          * 
          */
-        public void setSize(int minWidth, int maxWidth, int minHeight, int maxHeight, float newWidth, float newHeight)
+        public void setSize(final int minWidth, final int maxWidth, final int minHeight, final int maxHeight, final float newWidth, final float newHeight)
         {
                 setSizeRestrictions(minWidth, maxWidth, minHeight, maxHeight);
@@ -817 +832 @@
                 
                 // Remember current isFloating() values
-                boolean vfloating[] = new boolean[] { _d1.isFloating(), _d2.isFloating(), _d3.isFloating(), _d4.isFloating() };
+                final boolean vfloating[] = new boolean[] { _d1.isFloating(), _d2.isFloating(), _d3.isFloating(), _d4.isFloating() };
 
                 _d1.setFloating(true);
@@ -824 +839 @@
                 _d4.setFloating(true);
 
-                float xr = _d1.getX() + width;
-                float yb = _d1.getY() + height;
+                final float xr = _d1.getX() + width;
+                final float yb = _d1.getY() + height;
 
                 _d2.setX(xr);
@@ -841 +856 @@
         }
 
-    public void setAnchorCorners(Integer left, Integer right, Integer top, Integer bottom)
+    public void setAnchorCorners(final Integer left, final Integer right, final Integer top, final Integer bottom)
     {
         setAnchorLeft(left);
@@ -850 +865 @@
     
 
-        public void setPosition(int x, int y)
-        {
-                if (x == getX() && y == getY()) return;
+        public void setPosition(final int x, final int y)
+        {
+                if (x == getX() && y == getY()) {
+                        return;
+                }
 
                 // Remember current isFloating() values
-                boolean vfloating[] = new boolean[] { _d1.isFloating(), _d2.isFloating(), _d3.isFloating(), _d4.isFloating() };
-
-                int width = getWidth();
-                int height = getHeight();
+                final boolean vfloating[] = new boolean[] { _d1.isFloating(), _d2.isFloating(), _d3.isFloating(), _d4.isFloating() };
+
+                final int width = getWidth();
+                final int height = getHeight();
 
                 invalidateLink();
@@ -895 +912 @@
          * @return False if need to call super.setPosition
          */
-        public boolean setPositions(WidgetCorner src, float x, float y)
-        {
-                if (_settingPositionFlag) return false;
+        public boolean setPositions(final WidgetCorner src, final float x, final float y)
+        {
+                if (_settingPositionFlag) {
+                        return false;
+                }
                 
                 _settingPositionFlag = true;
@@ -904 +923 @@
 
                 // Check to see if the widget is fully being picked up
-                boolean isAllPickedUp = (_d1.isFloating() && _d2.isFloating() && _d3.isFloating() && _d4.isFloating());
+                final boolean isAllPickedUp = (_d1.isFloating() && _d2.isFloating() && _d3.isFloating() && _d4.isFloating());
 
                 // If so, then this will be called one by one ..
@@ -942 +961 @@
                                 }
 
-                                if (!(src == _d4 && _d1.isFloating() && _d4.isFloating()))
+                                if (!(src == _d4 && _d1.isFloating() && _d4.isFloating())) {
                                         _d1.setX(newX);
-                                if (!(src == _d1 && _d4.isFloating() && _d1.isFloating()))
+                                }
+                                if (!(src == _d1 && _d4.isFloating() && _d1.isFloating())) {
                                         _d4.setX(newX);
+                                }
 
                         } else if (src == _d2 || src == _d3) {
@@ -962 +983 @@
                                 }
 
-                                if (!(src == _d3 && _d2.isFloating() && _d3.isFloating()))
+                                if (!(src == _d3 && _d2.isFloating() && _d3.isFloating())) {
                                         _d2.setX(newX);
-                                if (!(src == _d2 && _d3.isFloating() && _d2.isFloating()))
+                                }
+                                if (!(src == _d2 && _d3.isFloating() && _d2.isFloating())) {
                                         _d3.setX(newX);
+                                }
                         }
 
@@ -986 +1009 @@
                                 }
 
-                                if (!(src == _d2 && _d1.isFloating() && _d2.isFloating()))
+                                if (!(src == _d2 && _d1.isFloating() && _d2.isFloating())) {
                                         _d1.setY(newY);
-                                if (!(src == _d1 && _d2.isFloating() && _d1.isFloating()))
+                                }
+                                if (!(src == _d1 && _d2.isFloating() && _d1.isFloating())) {
                                         _d2.setY(newY);
+                                }
 
                         } else if (src == _d3 || src == _d4) {
@@ -1006 +1031 @@
                                 }
 
-                                if (!(src == _d4 && _d3.isFloating() && _d4.isFloating()))
+                                if (!(src == _d4 && _d3.isFloating() && _d4.isFloating())) {
                                         _d3.setY(newY);
-                                if (!(src == _d3 && _d4.isFloating() && _d3.isFloating()))
+                                }
+                                if (!(src == _d3 && _d4.isFloating() && _d3.isFloating())) {
                                         _d4.setY(newY);
+                                }
                         }
                 }
 
                 // Update source text position so position is remembered from loading
-                float newTextX = getX();
-                float newTextY = getY();
+                final float newTextX = getX();
+                final float newTextY = getY();
                 if (_textRepresentation.getX() != newTextX || _textRepresentation.getY() != newTextY) {
                         _textRepresentation.setPosition(newTextX, newTextY);
@@ -1072 +1099 @@
         }
 
-        public final void onParentStateChanged(ItemParentStateChangedEvent e)
+        public final void onParentStateChanged(final ItemParentStateChangedEvent e)
         {
                 switch (e.getEventType()) {
@@ -1104 +1131 @@
          * 
          */
-        protected void onParentStateChanged(int eventType)
+        protected void onParentStateChanged(final int eventType)
         {
         }
@@ -1152 +1179 @@
         protected void paintInFreeSpace()
         {
-                GraphicsManager g = EcosystemManager.getGraphicsManager();
+                final GraphicsManager g = EcosystemManager.getGraphicsManager();
                 g.drawRectangle(new Point(getX(), getY()), new Dimension(getWidth(), getHeight()), 0.0, new Fill(FREESPACE_BACKCOLOR), null, null, null);
         }
@@ -1210 +1237 @@
         protected boolean removeSelf()
         {
-                Frame parent = getParentFrame();
+                final Frame parent = getParentFrame();
 
                 if (parent != null) {
@@ -1243 +1270 @@
         protected void invalidateSelf()
         {
-                AxisAlignedBoxBounds dirty = new AxisAlignedBoxBounds(getX(), getY(), getWidth(), getHeight());
+                final AxisAlignedBoxBounds dirty = new AxisAlignedBoxBounds(getX(), getY(), getWidth(), getHeight());
                 DisplayController.invalidateArea(dirty);
                 invalidateLink();
@@ -1254 +1281 @@
         {
                 if (_textRepresentation.getLink() != null || _textRepresentation.hasAction()) {
-                        AxisAlignedBoxBounds linkArea = _textRepresentation.getLinkDrawArea(getLinkX(), getLinkY());
+                        final AxisAlignedBoxBounds linkArea = _textRepresentation.getLinkDrawArea(getLinkX(), getLinkY());
                         DisplayController.invalidateArea(linkArea);
                 }
@@ -1316 +1343 @@
          *             If tag is empty.
          */
-        protected String getStrippedDataString(String tag)
-        {
-                if (tag == null) throw new NullPointerException("tag");
-                if (tag.length() == 0) throw new IllegalArgumentException("tag is empty");
+        protected String getStrippedDataString(final String tag)
+        {
+                if (tag == null) {
+                        throw new NullPointerException("tag");
+                }
+                if (tag.length() == 0) {
+                        throw new IllegalArgumentException("tag is empty");
+                }
 
                 if (getCurrentRepresentation().getData() != null) {
-                        for (String str : getCurrentRepresentation().getData()) {
+                        for (final String str : getCurrentRepresentation().getData()) {
                                 if (str != null && str.startsWith(tag) && str.length() > tag.length()) {
                                         return str.substring(tag.length());
@@ -1357 +1388 @@
          * 
          */
-        protected Integer getStrippedDataInt(String tag, Integer defaultValue)
+        protected Integer getStrippedDataInt(final String tag, final Integer defaultValue)
         {
                 String strippedStr = getStrippedDataString(tag);
@@ -1366 +1397 @@
                                 try {
                                         return Integer.parseInt(strippedStr);
-                                } catch (NumberFormatException e) { /* Consume */
+                                } catch (final NumberFormatException e) { /* Consume */
                                 }
                         }
@@ -1402 +1433 @@
          * 
          */
-        protected Long getStrippedDataLong(String tag, Long defaultValue)
+        protected Long getStrippedDataLong(final String tag, final Long defaultValue)
         {
                 String strippedStr = getStrippedDataString(tag);
@@ -1411 +1442 @@
                                 try {
                                         return Long.parseLong(strippedStr);
-                                } catch (NumberFormatException e) { /* Consume */
+                                } catch (final NumberFormatException e) { /* Consume */
                                 }
                         }
@@ -1446 +1477 @@
          * 
          */
-        protected Boolean getStrippedDataBool(String tag, Boolean defaultValue) {
+        protected Boolean getStrippedDataBool(final String tag, final Boolean defaultValue) {
 
                 String strippedStr = getStrippedDataString(tag);
@@ -1475 +1506 @@
          * 
          */
-        protected void removeData(String tag)
+        protected void removeData(final String tag)
         {
                 updateData(tag, null);
@@ -1483 +1514 @@
         protected void addDataIfCaseInsensitiveNotExists(String tag)
         {
-                if (tag == null) throw new NullPointerException("tag");
+                if (tag == null) {
+                        throw new NullPointerException("tag");
+                }
                 
                 List<String> data = getCurrentRepresentation().getData();
@@ -1491 +1524 @@
                 }
                 
-                for (String s : data) {
+                for (final String s : data) {
                         if (s != null && s.equalsIgnoreCase(tag)) {
                                 return;
@@ -1525 +1558 @@
          * 
          */
-        protected void updateData(String tag, String newData)
-        {
-                if (tag == null)
+        protected void updateData(final String tag, final String newData)
+        {
+                if (tag == null) {
                         throw new NullPointerException("tag");
-                else if (tag.length() == 0)
+                } else if (tag.length() == 0) {
                         throw new IllegalArgumentException("tag is empty");
+                }
 
                 // Get current data
@@ -1537 +1571 @@
                 if (data != null) {
                         for (int i = 0; i < data.size(); i++) {
-                                String str = data.get(i);
+                                final String str = data.get(i);
                                 if (str != null && str.startsWith(tag)) {
                                         data.remove(i);
@@ -1545 +1579 @@
 
                 if (newData != null) {
-                        if (data != null)
+                        if (data != null) {
                                 data.add(newData);
-                        else {
+                        } else {
                                 data = new LinkedList<String>();
                                 data.add(newData);
@@ -1556 +1590 @@
         }
         
-        public boolean containsData(String str) {
-                assert(str != null);
-                if (getCurrentRepresentation().getData() != null) 
-                        return getCurrentRepresentation().getData().contains(str);
-                return false;
-        }
-        
-        public boolean containsDataTrimmedIgnoreCase(String str) {
+        public boolean containsData(final String str) {
                 assert(str != null);
                 if (getCurrentRepresentation().getData() != null) {
-                        for (String data : getCurrentRepresentation().getData()) {
+                        return getCurrentRepresentation().getData().contains(str);
+                }
+                return false;
+        }
+        
+        public boolean containsDataTrimmedIgnoreCase(final String str) {
+                assert(str != null);
+                if (getCurrentRepresentation().getData() != null) {
+                        for (final String data : getCurrentRepresentation().getData()) {
                                 if (data != null && data.trim().equalsIgnoreCase(str)) {
                                         return true;
@@ -1586 +1621 @@
          *                      a text item.
          */
-        public void setLink(String link, Text linker)
+        public void setLink(final String link, final Text linker)
         {
                 // Make sure the link is redrawn when a link is added
-                if (link == null) invalidateLink();
+                if (link == null) {
+                        invalidateLink();
+                }
                 getSource().setLink(link);
-                if (link != null) invalidateLink();
-        }
-
-        public void setBackgroundColor(Colour c)
+                if (link != null) {
+                        invalidateLink();
+                }
+        }
+
+        public void setBackgroundColor(final Colour c)
         {
                 getSource().setBackgroundColor(c);
@@ -1622 +1661 @@
                 // a parent ... thus must manually format link
 
-                String link = getLink();
-
-                if (link == null || link.length() == 0)
+                final String link = getLink();
+
+                if (link == null || link.length() == 0) {
                         return null;
+                }
 
                 if (FrameIO.isPositiveInteger(link)) { // relative - convert to
@@ -1632 +1672 @@
                         // Get the frameset of this item
                         Frame parent = getParentFrame();
-                        if (parent == null)
+                        if (parent == null) {
                                 parent = DisplayController.getCurrentFrame();
-                        if (parent == null)
+                        }
+                        if (parent == null) {
                                 return null;
-
-                        String framesetName = parent.getFramesetName();
-
-                        if (framesetName == null)
+                        }
+
+                        final String framesetName = parent.getFramesetName();
+
+                        if (framesetName == null) {
                                 return null;
+                        }
 
                         return framesetName + link;
@@ -1659 +1702 @@
          *              The color to set.
          */
-        public void setWidgetEdgeColor(Colour c)
+        public void setWidgetEdgeColor(final Colour c)
         {
                 // Indirectly invokes setSourceBorderColor accordingly
-                for (Item i : _expediteeItems) i.setColor(c);
+                for (final Item i : _expediteeItems) {
+                        i.setColor(c);
+                }
         }
         
@@ -1673 +1718 @@
          *              The new thickness to set.
          */
-        public void setWidgetEdgeThickness(float thickness) {
+        public void setWidgetEdgeThickness(final float thickness) {
                 _l1.setThickness(thickness, true);
                 //for (Item i : _expediteeItems) i.setThickness(thickness);
@@ -1688 +1733 @@
 
         // TODO: Maybe rename setSource* ..  to update* ... These should actually be friendly!
-        public void setSourceColor(Colour c) {
+        public void setSourceColor(final Colour c) {
                 _textRepresentation.setColor(c);
         }
 
-        public void setSourceBorderColor(Colour c) {
+        public void setSourceBorderColor(final Colour c) {
                 _textRepresentation.setBorderColor(c);
         }
 
-        public void setSourceFillColor(Colour c) {
+        public void setSourceFillColor(final Colour c) {
                 _textRepresentation.setFillColor(c);
         }
 
-        public void setSourceThickness(float newThickness, boolean setConnected) {
+        public void setSourceThickness(final float newThickness, final boolean setConnected) {
                 _textRepresentation.setThickness(newThickness, setConnected);
         }
 
-        public void setSourceData(List<String> data) {
+        public void setSourceData(final List<String> data) {
                 _textRepresentation.setData(data);
         }
@@ -1716 +1761 @@
         }
 
-        public void setAnchorLeft(Integer anchor)
+        public void setAnchorLeft(final Integer anchor)
         {
                 _anchoring.setLeftAnchor(anchor);
@@ -1732 +1777 @@
         }
 
-        public void setAnchorRight(Integer anchor) {
+        public void setAnchorRight(final Integer anchor) {
                 _anchoring.setRightAnchor(anchor);
                 // Anchor right-edge corners (dots) as well
@@ -1749 +1794 @@
         }
         
-        public void setAnchorTop(Integer anchor) {
+        public void setAnchorTop(final Integer anchor) {
                 _anchoring.setTopAnchor(anchor);
                 // Anchor top-edge corners (dots) as well
@@ -1764 +1809 @@
         }
 
-        public void setAnchorBottom(Integer anchor) {
+        public void setAnchorBottom(final Integer anchor) {
                 _anchoring.setBottomAnchor(anchor);
                 // Anchor bottom-edge corners (dots) as well
@@ -1837 +1882 @@
         public Clip getClip()
         {
-                Clip clip = new Clip(getBounds());
+                final Clip clip = new Clip(getBounds());
                 return clip;
         }