Changeset 8003

Timestamp:

2004-08-19T16:13:45+12:00 (20 years ago)

Author:

mdewsnip

Message:

Tightened up some public functions to be private.

Location:

trunk/gli/src/org/greenstone/gatherer/gui

Files:

• GComboBox.java (modified) (2 diffs)
• GUIManager.java (modified) (6 diffs)
• LongProgressBar.java (modified) (1 diff)
• TransformCharacterTextField.java (modified) (3 diffs)
• metaaudit/Autofilter.java (modified) (1 diff)

trunk/gli/src/org/greenstone/gatherer/gui/GComboBox.java

@@ -150 +150 @@
     }
 
-    public void setBackgroundEditableColor(Color editable_background) {
-    this.editable_background = editable_background;
-    setEditor(new Editor());
-    setRenderer(new Renderer());
-    }
+//      public void setBackgroundEditableColor(Color editable_background) {
+//      this.editable_background = editable_background;
+//      setEditor(new Editor());
+//      setRenderer(new Renderer());
+//      }
 
     public void setBackgroundNonSelectionColor(Color background) {
@@ -184 +184 @@
     }
 
-    public void setTextEditableColor(Color editable_foreground) {
-    this.editable_foreground = editable_foreground;
-    setEditor(new Editor());
-    setRenderer(new Renderer());
-    }
+//      public void setTextEditableColor(Color editable_foreground) {
+//      this.editable_foreground = editable_foreground;
+//      setEditor(new Editor());
+//      setRenderer(new Renderer());
+//      }
 
     public void setTextNonSelectionColor(Color foreground) {

trunk/gli/src/org/greenstone/gatherer/gui/GUIManager.java

@@ -536 +536 @@
     }
 
-    public Component getSelectedView() {
-    return tab_pane.getSelectedComponent();
-    }
+//      public Component getSelectedView() {
+//      return tab_pane.getSelectedComponent();
+//      }
     /** This method is called when the collection is being built, and is used to disable all controls in all pane which could change the state of the collection.
      */
@@ -615 +615 @@
 
     /** Specifies whether a certain tab is enabled or not. */
-    public void setTabEnabled(String rawname, boolean state) {
+    private void setTabEnabled(String rawname, boolean state) {
     // Retrieve the dictionary based name.
     String name = Dictionary.get("GUI." + rawname);
@@ -682 +682 @@
     /** When the edit metadata option is choosen from the menu, this method is called to ensure we only edit the metadata if there is metadata loaded.
      */
-    public void showEditMetadataBox() {
+    private void showEditMetadataBox() {
     if(Gatherer.c_man.getCollection() != null) {
         Gatherer.c_man.getCollection().msm.editMDS(null, MetadataEditorManager.NORMAL);
@@ -689 +689 @@
     /** When the load collection option is choosen this method is called to produce the modal file load prompt.
      */
-    public boolean showLoadCollectionBox() {
+    private boolean showLoadCollectionBox() {
     boolean result = false;
     // We first try the simple open collection dialog
@@ -759 +759 @@
     /** This method is used to open the new collection box on the screen.
      */
-    public void showNewCollectionPrompt() {
+    private void showNewCollectionPrompt() {
     NewCollectionMetadataPrompt ncm_prompt = null;
     // Create the collection details prompt from new collection prompt
@@ -829 +829 @@
      * @return A <i>boolean</i> which is <i>true</i> if the collection was saved successfully, <i>false</i> otherwise.
      */
-    public boolean showSaveCollectionBox(boolean close_after, boolean exit_after) {
+    private boolean showSaveCollectionBox(boolean close_after, boolean exit_after) {
     //SaveCollectionBox save_collection_box = new SaveCollectionBox();
     //Rectangle bounds = save_collection_box.getBounds();

trunk/gli/src/org/greenstone/gatherer/gui/LongProgressBar.java

@@ -141 +141 @@
 
     /** This method is an unsafe way to set the progress bar to be indeterminate, -unless- it is called from the Event Queue. */
-    public void unsafeSetIndeterminate(boolean state) {
+    private void unsafeSetIndeterminate(boolean state) {
     super.setIndeterminate(state);
     }
 
-    public void unsafeSetString(String label) {
+    private void unsafeSetString(String label) {
     super.setString(label);
     }
 
-    public void unsafeSetValue(int value) {
+    private void unsafeSetValue(int value) {
     super.setValue(value);
     }

trunk/gli/src/org/greenstone/gatherer/gui/TransformCharacterTextField.java

@@ -63 +63 @@
     super(text);
     }
- 
-    public void blockCharacter(char x) {
-    mappings.put(new Character(x), null);
-    }
+
+//      public void blockCharacter(char x) {
+//      mappings.put(new Character(x), null);
+//      }
 
     public void replaceCharacter(char x, char y) {
@@ -72 +72 @@
     }
 
-    public void restoreCharacter(char x) {
-    mappings.remove(new Character(x));
-    }
+//      public void restoreCharacter(char x) {
+//      mappings.remove(new Character(x));
+//      }
 
     public void setBlockAllExceptMappings(boolean block_all_except_mappings) {
@@ -82 +82 @@
 
     protected Document createDefaultModel() {
-    return new TransformCharacterTextDocument();
+    return new TransformCharacterTextDocument();
     }
- 
+
     private class TransformCharacterTextDocument 
     extends PlainDocument {

trunk/gli/src/org/greenstone/gatherer/gui/metaaudit/Autofilter.java

@@ -82 +82 @@
      * @return -1 if s is less than p, 0 if they are equal, 1 if s is greater than p
      */
-    static public int compareToPattern(String s, String p) {
+    static private int compareToPattern(String s, String p) {
     if(p.indexOf(StaticStrings.STAR_CHAR) == -1) {
         return compareToPattern(s, p, 0, 0, false);