Changeset 1498 for trunk/src/org/expeditee/settings/Settings.java

Timestamp:

12/05/19 17:23:45 (4 years ago)

Author:

bnemhaus

Message:

Refactored variable name for clarity.

Comments explaining why an exception is being suppressed.

File:

• trunk/src/org/expeditee/settings/Settings.java (modified) (5 diffs)

trunk/src/org/expeditee/settings/Settings.java

@@ -254 +254 @@
         private static void generateSettingsTree(String page, Text text) {
                 Frame parent = text.getParentOrCurrentFrame();
-                FrameCreator frames = new FrameCreator(parent.getFramesetName(), parent.getPath(), page, FrameCreator.ExistingFramesetOptions.AppendSegregatedFrames, false, null);
-                text.setLink(frames.getName());
+                FrameCreator frameCreator = new FrameCreator(parent.getFramesetName(), parent.getPath(), page, FrameCreator.ExistingFramesetOptions.AppendSegregatedFrames, false, null);
+                text.setLink(frameCreator.getName());
                 
                 // Add subpages of the current page by recursing
@@ -265 +265 @@
                                 }
                                 System.out.println("@Settings: Generating " + name);
-                                generateSettingsTree(k, frames.addText(name.substring(0, 1).toUpperCase() + name.substring(1), null, null, null, false));
+                                generateSettingsTree(k, frameCreator.addText(name.substring(0, 1).toUpperCase() + name.substring(1), null, null, null, false));
                         }
                 }
                 
                 // Start building current page
-                frames.setLastY(frames.getLastY() + 20);
+                frameCreator.setLastY(frameCreator.getLastY() + 20);
                 //      Add Settings of the current page
                 PageDescriptor pd = _pages.get(page);
                 if (pd == null) {
-                        frames.save();
+                        frameCreator.save();
                         return;
                 }
@@ -290 +290 @@
                         int x = 0, y = 0;
                         
-                        Frame currentFrame = frames.getCurrentFrame();
+                        Frame currentFrame = frameCreator.getCurrentFrame();
                         if (key.toLowerCase().equals("pass")) {
                                 // Special case for Password widgets
-                                Text passwordWidgetText = frames.addText("iw: org.expeditee.items.widgets.Passwsord", null, null, null, false);
+                                Text passwordWidgetText = frameCreator.addText("iw: org.expeditee.items.widgets.Passwsord", null, null, null, false);
                                 Password pw = new Password(passwordWidgetText, null);
                                 pw.setPassword("");
@@ -317 +317 @@
                                 }
                                 settingRepresentation.setID(currentFrame.getNextItemID());
-                                frames.addItem(settingRepresentation, false);                           
+                                frameCreator.addItem(settingRepresentation, false);                             
                                 x = settingRepresentation.getX() + settingRepresentation.getBoundsWidth();
                                 y = settingRepresentation.getY();
@@ -326 +326 @@
                         Text tooltip = currentFrame.addText(x, y, "// " + setting.getTooltip(), null);
                         tooltip.rebuild(true);
-                        if (tooltip.getY() + tooltip.getBoundsHeight() > frames.getLastY()) {
-                                frames.setLastY(tooltip.getY() + tooltip.getBoundsHeight());
-                        }
-                }
-                
-                frames.save();
+                        if (tooltip.getY() + tooltip.getBoundsHeight() > frameCreator.getLastY()) {
+                                frameCreator.setLastY(tooltip.getY() + tooltip.getBoundsHeight());
+                        }
+                }
+                
+                frameCreator.save();
         }
 }