OSDN Git Service

* javax/swing/text/html/parser/ParserDelegator.java (callBack,
authortromey <tromey@138bc75d-0d04-0410-961f-82ee72b054a4>
Tue, 26 Apr 2005 18:57:40 +0000 (18:57 +0000)
committertromey <tromey@138bc75d-0d04-0410-961f-82ee72b054a4>
Tue, 26 Apr 2005 18:57:40 +0000 (18:57 +0000)
gnu): Now package-private.
* javax/swing/text/html/parser/DocumentParser.java (parser,
callBack, gnu): Now package-private.
* javax/swing/text/StringContent.java (content, setOffset,
checkLocation): Now package-private.
* javax/swing/text/JTextComponent.java (doc): Now
package-private.
* javax/swing/plaf/basic/BasicToolBarUI.java (owner,
lastGoodOrientation, origParent, borders, cachedBounds,
cachedOrientation, DragWindow): Now package-private.
* javax/swing/plaf/basic/BasicTabbedPaneUI.java (incrButton,
decrButton, viewport, panel, currentScrollLocation, layoutManager,
tabAreaRect, contentRect, createIncreaseButton,
createDecreaseButton, findPointForIndex): Now package-private.
* javax/swing/plaf/basic/BasicSplitPaneDivider.java
(currentDividerLocation, moveDividerTo): Now package-private.
* javax/swing/plaf/basic/BasicSliderUI.java (findClosestTick): Now
package-private.
* javax/swing/plaf/basic/BasicScrollBarUI.java
(calculatePreferredSize, valueForYPosition, valueForXPosition):
Now package-private.
* javax/swing/plaf/basic/BasicPopupMenuUI.java
(topWindowListener): Now package-private.
* javax/swing/plaf/basic/BasicOptionPaneUI.java (iconSize,
OK_STRING, YES_STRING, NO_STRING, CANCEL_STRING): Now constants.
(messageAreaContainer, buttonContainer, resetSelectedValue): Now
package-private.
* javax/swing/plaf/basic/BasicListUI.java (damageLayout): Now
package-private.
* javax/swing/plaf/basic/BasicInternalFrameTitlePane.java (title):
Now package-private.
* javax/swing/plaf/basic/BasicDesktopIconUI.java (button): Now
package-private.
* javax/swing/plaf/basic/BasicComboBoxUI.java (largestItemSize,
borderInsets): Now package-private.
(arrowButtonWidth): Likewise.  Now a constant.
* javax/swing/plaf/basic/BasicColorChooserUI.java (chooser, pane,
makeTabs, updatePreviewPanel): Now package-private.
* javax/swing/plaf/basic/BasicArrowButton.java (shadow,
darkShadow, highlight): Now package-private.
* javax/swing/colorchooser/DefaultSwatchChooserPanel.java
(addColorToQueue): Now package-private.
* javax/swing/colorchooser/DefaultRGBChooserPanel.java
(spinnerChange, sliderChange, updateChange, R, G, B, RSpinner,
GSpinner, BSpinner): Now package-private.
* javax/swing/colorchooser/DefaultHSBChooserPanel.java
(gradientImage, trackImage, slider, hRadio, sRadio, bRadio,
hSpinner, sSpinner, bSpinner, gradientPoint, internalChange,
spinnerTrigger, locked, handlingMouse, updateImage, updateSlider,
updateTrack): Now package-private.
* javax/swing/TransferHandler.java (clipboard): Now
package-private.
* javax/swing/ToolTipManager.java (showTip, hideTip): Now
package-private.
* javax/swing/Timer.java (drainEvents, queueEvent): Now
package-private.
* javax/swing/RepaintManager.java (globalManager): Now
package-private.
* javax/swing/JFormattedTextField.java (editValid): Now
package-private.
* javax/swing/JColorChooser.java (makeModal): Now
package-private.
* java/awt/geom/GeneralPath.java (rule, types, xpoints, ypoints,
index, path): Now package-private.
* java/awt/geom/Area.java (solids, holes, cubicCubicIntersect,
lineQuadIntersect, lineCubicIntersect, linesIntersect,
pointEquals): Now package-private.
* java/awt/geom/Arc2D.java (type): Now package-private.
* java/awt/Window.java (windowFocusOwner): Now package-private.
* java/awt/TextComponent.java (editable, selectionStart,
selectionEnd, text, getIndexAtPoint, getCharacterBounds): Now
package-private.
* java/awt/MenuItem.java (actionCommand, enabled, label): Now
package-private.
* java/awt/MenuComponent.java (focusListener): Now
package-private.
* java/awt/Frame.java (state): Now package-private.
* java/awt/Choice.java (pItems): Now package-private.
* java/awt/Checkbox.java (state): Now package-private.
* java/awt/Button.java (actionCommand, label): Now
package-private.
* gnu/javax/swing/text/html/parser/support/Parser.java
(attributes, _handleEndTag_remaining, _handleStartTag): Now
package-private.
* gnu/java/beans/decoder/PersistenceParser.java (javaHandler): Now
package-private.
* gnu/java/awt/peer/gtk/GdkGraphics2D.java (paint, stroke, fg, bg,
clip, transform, font, comp): Now package-private.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@98786 138bc75d-0d04-0410-961f-82ee72b054a4

41 files changed:
libjava/ChangeLog
libjava/gnu/java/awt/peer/gtk/GdkGraphics2D.java
libjava/gnu/java/beans/decoder/PersistenceParser.java
libjava/gnu/javax/swing/text/html/parser/support/Parser.java
libjava/java/awt/Button.java
libjava/java/awt/Checkbox.java
libjava/java/awt/Choice.java
libjava/java/awt/Frame.java
libjava/java/awt/MenuComponent.java
libjava/java/awt/MenuItem.java
libjava/java/awt/TextComponent.java
libjava/java/awt/Window.java
libjava/java/awt/geom/Arc2D.java
libjava/java/awt/geom/Area.java
libjava/java/awt/geom/GeneralPath.java
libjava/javax/swing/JColorChooser.java
libjava/javax/swing/JFormattedTextField.java
libjava/javax/swing/RepaintManager.java
libjava/javax/swing/Timer.java
libjava/javax/swing/ToolTipManager.java
libjava/javax/swing/TransferHandler.java
libjava/javax/swing/colorchooser/DefaultHSBChooserPanel.java
libjava/javax/swing/colorchooser/DefaultRGBChooserPanel.java
libjava/javax/swing/colorchooser/DefaultSwatchChooserPanel.java
libjava/javax/swing/plaf/basic/BasicArrowButton.java
libjava/javax/swing/plaf/basic/BasicColorChooserUI.java
libjava/javax/swing/plaf/basic/BasicComboBoxUI.java
libjava/javax/swing/plaf/basic/BasicDesktopIconUI.java
libjava/javax/swing/plaf/basic/BasicInternalFrameTitlePane.java
libjava/javax/swing/plaf/basic/BasicListUI.java
libjava/javax/swing/plaf/basic/BasicOptionPaneUI.java
libjava/javax/swing/plaf/basic/BasicPopupMenuUI.java
libjava/javax/swing/plaf/basic/BasicScrollBarUI.java
libjava/javax/swing/plaf/basic/BasicSliderUI.java
libjava/javax/swing/plaf/basic/BasicSplitPaneDivider.java
libjava/javax/swing/plaf/basic/BasicTabbedPaneUI.java
libjava/javax/swing/plaf/basic/BasicToolBarUI.java
libjava/javax/swing/text/JTextComponent.java
libjava/javax/swing/text/StringContent.java
libjava/javax/swing/text/html/parser/DocumentParser.java
libjava/javax/swing/text/html/parser/ParserDelegator.java

index 519f9ae..d32eb9e 100644 (file)
@@ -1,5 +1,97 @@
 2005-04-26  Tom Tromey  <tromey@redhat.com>
 
+       * javax/swing/text/html/parser/ParserDelegator.java (callBack,
+       gnu): Now package-private.
+       * javax/swing/text/html/parser/DocumentParser.java (parser,
+       callBack, gnu): Now package-private.
+       * javax/swing/text/StringContent.java (content, setOffset,
+       checkLocation): Now package-private.
+       * javax/swing/text/JTextComponent.java (doc): Now
+       package-private.
+       * javax/swing/plaf/basic/BasicToolBarUI.java (owner,
+       lastGoodOrientation, origParent, borders, cachedBounds,
+       cachedOrientation, DragWindow): Now package-private.
+       * javax/swing/plaf/basic/BasicTabbedPaneUI.java (incrButton,
+       decrButton, viewport, panel, currentScrollLocation, layoutManager,
+       tabAreaRect, contentRect, createIncreaseButton,
+       createDecreaseButton, findPointForIndex): Now package-private.
+       * javax/swing/plaf/basic/BasicSplitPaneDivider.java
+       (currentDividerLocation, moveDividerTo): Now package-private.
+       * javax/swing/plaf/basic/BasicSliderUI.java (findClosestTick): Now
+       package-private.
+       * javax/swing/plaf/basic/BasicScrollBarUI.java
+       (calculatePreferredSize, valueForYPosition, valueForXPosition):
+       Now package-private.
+       * javax/swing/plaf/basic/BasicPopupMenuUI.java
+       (topWindowListener): Now package-private.
+       * javax/swing/plaf/basic/BasicOptionPaneUI.java (iconSize,
+       OK_STRING, YES_STRING, NO_STRING, CANCEL_STRING): Now constants.
+       (messageAreaContainer, buttonContainer, resetSelectedValue): Now
+       package-private.
+       * javax/swing/plaf/basic/BasicListUI.java (damageLayout): Now
+       package-private.
+       * javax/swing/plaf/basic/BasicInternalFrameTitlePane.java (title):
+       Now package-private.
+       * javax/swing/plaf/basic/BasicDesktopIconUI.java (button): Now
+       package-private.
+       * javax/swing/plaf/basic/BasicComboBoxUI.java (largestItemSize,
+       borderInsets): Now package-private.
+       (arrowButtonWidth): Likewise.  Now a constant.
+       * javax/swing/plaf/basic/BasicColorChooserUI.java (chooser, pane,
+       makeTabs, updatePreviewPanel): Now package-private.
+       * javax/swing/plaf/basic/BasicArrowButton.java (shadow,
+       darkShadow, highlight): Now package-private.
+       * javax/swing/colorchooser/DefaultSwatchChooserPanel.java
+       (addColorToQueue): Now package-private.
+       * javax/swing/colorchooser/DefaultRGBChooserPanel.java
+       (spinnerChange, sliderChange, updateChange, R, G, B, RSpinner,
+       GSpinner, BSpinner): Now package-private.
+       * javax/swing/colorchooser/DefaultHSBChooserPanel.java
+       (gradientImage, trackImage, slider, hRadio, sRadio, bRadio,
+       hSpinner, sSpinner, bSpinner, gradientPoint, internalChange,
+       spinnerTrigger, locked, handlingMouse, updateImage, updateSlider,
+       updateTrack): Now package-private.
+       * javax/swing/TransferHandler.java (clipboard): Now
+       package-private.
+       * javax/swing/ToolTipManager.java (showTip, hideTip): Now
+       package-private.
+       * javax/swing/Timer.java (drainEvents, queueEvent): Now
+       package-private.
+       * javax/swing/RepaintManager.java (globalManager): Now
+       package-private.
+       * javax/swing/JFormattedTextField.java (editValid): Now
+       package-private.
+       * javax/swing/JColorChooser.java (makeModal): Now
+       package-private.
+       * java/awt/geom/GeneralPath.java (rule, types, xpoints, ypoints,
+       index, path): Now package-private.
+       * java/awt/geom/Area.java (solids, holes, cubicCubicIntersect,
+       lineQuadIntersect, lineCubicIntersect, linesIntersect,
+       pointEquals): Now package-private.
+       * java/awt/geom/Arc2D.java (type): Now package-private.
+       * java/awt/Window.java (windowFocusOwner): Now package-private.
+       * java/awt/TextComponent.java (editable, selectionStart,
+       selectionEnd, text, getIndexAtPoint, getCharacterBounds): Now
+       package-private.
+       * java/awt/MenuItem.java (actionCommand, enabled, label): Now
+       package-private.
+       * java/awt/MenuComponent.java (focusListener): Now
+       package-private.
+       * java/awt/Frame.java (state): Now package-private.
+       * java/awt/Choice.java (pItems): Now package-private.
+       * java/awt/Checkbox.java (state): Now package-private.
+       * java/awt/Button.java (actionCommand, label): Now
+       package-private.
+       * gnu/javax/swing/text/html/parser/support/Parser.java
+       (attributes, _handleEndTag_remaining, _handleStartTag): Now
+       package-private.
+       * gnu/java/beans/decoder/PersistenceParser.java (javaHandler): Now
+       package-private.
+       * gnu/java/awt/peer/gtk/GdkGraphics2D.java (paint, stroke, fg, bg,
+       clip, transform, font, comp): Now package-private.
+
+2005-04-26  Tom Tromey  <tromey@redhat.com>
+
        * java/awt/Robot.java (waitForIdle): Call invokeAndWait in a
        static way.
 
index 513c75c..6515247 100644 (file)
@@ -110,19 +110,22 @@ public class GdkGraphics2D extends Graphics2D
   
   private final int native_state = GtkGenericPeer.getUniqueInteger();  
 
-  private Paint paint;
-  private Stroke stroke;
-  private Color fg;
-  private Color bg;
-  private Shape clip;
-  private AffineTransform transform;
+  // These are package-private to avoid accessor methods.
+  Paint paint;
+  Stroke stroke;
+  Color fg;
+  Color bg;
+  Shape clip;
+  AffineTransform transform;
   private GtkComponentPeer component;
-  private Font font;
+  // This is package-private to avoid an accessor method.
+  Font font;
   private RenderingHints hints;
   private BufferedImage bimage;
   private boolean pixelConversionRequired;
   private int[] pixelBuffer;
-  private Composite comp;
+  // This is package-private to avoid an accessor method.
+  Composite comp;
   private Stack stateStack;
 
   private native void initState(GtkComponentPeer component);
index 3315478..5bb4acf 100644 (file)
@@ -1,5 +1,5 @@
 /* gnu.java.beans.PersistenceParser
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -83,8 +83,9 @@ public class PersistenceParser extends DefaultHandler implements Context
        private ElementHandler currentHandler;
 
        /** The real root element that stores all objects created during parsing.
+        * Package-private to avoid an accessor method.
         */
-       private JavaHandler javaHandler;
+       JavaHandler javaHandler;
 
        /** Stores the decoded objects. */
        private List objects = new LinkedList();
index 084edf6..1e5147a 100755 (executable)
@@ -157,8 +157,9 @@ public class Parser
 
   /**
    * The attributes of the current HTML element.
+   * Package-private to avoid an accessor method.
    */
-  private htmlAttributeSet attributes =
+  htmlAttributeSet attributes =
     htmlAttributeSet.EMPTY_HTML_ATTRIBUTE_SET;
 
   /**
@@ -1154,9 +1155,10 @@ public class Parser
 
   /**
    * Actions that are also required if the closing action was
-   * intiated by the tag validator.
+   * initiated by the tag validator.
+   * Package-private to avoid an accessor method.
    */
-  private void _handleEndTag_remaining(TagElement tag)
+  void _handleEndTag_remaining(TagElement tag)
   {
     HTML.Tag h = tag.getHTMLTag();
 
@@ -1183,9 +1185,10 @@ public class Parser
    * A hooks for operations, preceeding call to handleStartTag().
    * The method is called when the HTML opening tag ((like &lt;table&gt;)
    * is found.
+   * Package-private to avoid an accessor method.
    * @param The tag
    */
-  private void _handleStartTag(TagElement tag)
+  void _handleStartTag(TagElement tag)
   {
     validator.openTag(tag, attributes);
     startingTag(tag);
index eb0fe3b..a71a0d7 100644 (file)
@@ -77,13 +77,15 @@ private static final long serialVersionUID = -8774683716313001058L;
 
 /**
   * @serial The action command name for this button.
+  * This is package-private to avoid an accessor method.
   */
-private String actionCommand;
+String actionCommand;
 
 /**
   * @serial The label for this button.
+  * This is package-private to avoid an accessor method.
   */
-private String label;
+String label;
 
 // List of ActionListeners for this class.
 private transient ActionListener action_listeners;
index 2a8b62a..a0c86c1 100644 (file)
@@ -89,8 +89,9 @@ private String label;
 
 /**
   * @serial The state of this checkbox.
+  * This is package-private to avoid an accessor method.
   */
-private boolean state;
+boolean state;
 
 // The list of listeners for this object.
 private transient ItemListener item_listeners;
index c6a532a..cdfd704 100644 (file)
@@ -74,8 +74,9 @@ private static final long serialVersionUID = -4075310674757313071L;
 
 /**
   * @serial A list of items for the choice box, which can be <code>null</code>.
+  * This is package-private to avoid an accessor method.
   */
-private Vector pItems = new Vector();
+Vector pItems = new Vector();
 
 /**
   * @serial The index of the selected item in the choice box.
index d180984..77596ef 100644 (file)
@@ -186,8 +186,9 @@ private boolean resizable = true;
 /**
   * @serial The state of this frame.
   * // FIXME: What are the values here?
+  * This is package-private to avoid an accessor method.
   */
-private int state;
+int state;
 
 /**
   * @serial The title of the frame.
index f5677fe..3d41e40 100644 (file)
@@ -161,8 +161,9 @@ private static final long serialVersionUID = -4536902356223894379L;
    *
    * @see AccessibleAWTMenuComponent#addFocusListener(java.awt.event.FocusListener)
    * @serial ignored.
+   * This is package-private to avoid an accessor method.
    */
-  private transient FocusListener focusListener;
+  transient FocusListener focusListener;
 
 /*************************************************************************/
 
index 56082d3..f40f4af 100644 (file)
@@ -75,13 +75,15 @@ private static final long serialVersionUID = -21757335363267194L;
 
 /**
   * @serial The name of the action command generated by this item.
+  * This is package-private to avoid an accessor method.
   */
-private String actionCommand;
+String actionCommand;
 
 /**
   * @serial Indicates whether or not this menu item is enabled.
+  * This is package-private to avoid an accessor method.
   */
-private boolean enabled = true;
+boolean enabled = true;
 
 /**
   * @serial The mask of events that are enabled for this menu item.
@@ -90,8 +92,9 @@ long eventMask;
 
 /**
   * @serial This menu item's label
+  * This is package-private to avoid an accessor method.
   */
-private String label;
+String label;
 
 /**
   * @serial The shortcut for this menu item, if any
index 4c38439..34ea698 100644 (file)
@@ -76,23 +76,27 @@ private static final long serialVersionUID = -2214773872412987419L;
 
 /**
   * @serial Indicates whether or not this component is editable.
+  * This is package-private to avoid an accessor method.
   */
-private boolean editable;
+boolean editable;
 
 /**
   * @serial The starting position of the selected text region.
+  * This is package-private to avoid an accessor method.
   */
-private int selectionStart;
+int selectionStart;
 
 /**
   * @serial The ending position of the selected text region.
+  * This is package-private to avoid an accessor method.
   */
-private int selectionEnd;
+int selectionEnd;
 
 /**
   * @serial The text in the component
+  * This is package-private to avoid an accessor method.
   */
-private String text;
+String text;
 
 /**
   * A list of listeners that will receive events from this object.
@@ -708,8 +712,9 @@ paramString()
   
   /*******************************/
   // Provide AccessibleAWTTextComponent access to several peer functions that
-  // aren't publicly exposed.
-  private synchronized int
+  // aren't publicly exposed.  This is package-private to avoid an accessor
+  // method.
+  synchronized int
   getIndexAtPoint(Point p)
   {
     TextComponentPeer tcp = (TextComponentPeer)getPeer();
@@ -718,7 +723,7 @@ paramString()
     return -1;
   }
   
-  private synchronized Rectangle
+  synchronized Rectangle
   getCharacterBounds(int i)
   {
     TextComponentPeer tcp = (TextComponentPeer)getPeer();
index 0a39d20..e4e0a36 100644 (file)
@@ -90,7 +90,8 @@ public class Window extends Container implements Accessible
 
   private transient boolean shown;
 
-  private transient Component windowFocusOwner;
+  // This is package-private to avoid an accessor method.
+  transient Component windowFocusOwner;
   
   /*
    * The number used to generate the name returned by getName.
index 5ce3b08..2cdc170 100644 (file)
@@ -73,8 +73,9 @@ public abstract class Arc2D extends RectangularShape
    */
   public static final int PIE = 2;
 
-  /** The closure type of this arc. */
-  private int type;
+  /** The closure type of this arc.  This is package-private to avoid an
+   * accessor method.  */
+  int type;
 
   /**
    * Create a new arc, with the specified closure type.
index 68f905f..7825529 100644 (file)
@@ -89,13 +89,15 @@ public class Area implements Shape, Cloneable
 
   /**
    * Segment vectors containing solid areas and holes
+   * This is package-private to avoid an accessor method.
    */
-  private Vector solids;
+  Vector solids;
 
   /**
    * Segment vectors containing solid areas and holes
+   * This is package-private to avoid an accessor method.
    */
-  private Vector holes;
+  Vector holes;
 
   /**
    * Vector (temporary) storing curve-curve intersections
@@ -1315,9 +1317,10 @@ public class Area implements Shape, Cloneable
    * This is a reasonably accurate method, although the recursion depth
    * is typically around 20, the bounding-box tests allow for significant
    * pruning of the subdivision tree.
+   * 
+   * This is package-private to avoid an accessor method.
    */
-  private Intersection[] cubicCubicIntersect(CubicSegment curve1,
-                                             CubicSegment curve2)
+  Intersection[] cubicCubicIntersect(CubicSegment curve1, CubicSegment curve2)
   {
     Rectangle2D r1 = curve1.getBounds();
     Rectangle2D r2 = curve2.getBounds();
@@ -1349,8 +1352,9 @@ public class Area implements Shape, Cloneable
    * Or null if no intersections are found1
    * This is done through combining the line's equation with the
    * parametric form of the Bezier and solving the resulting quadratic.
+   * This is package-private to avoid an accessor method.
    */
-  private Intersection[] lineQuadIntersect(LineSegment l, QuadSegment c)
+  Intersection[] lineQuadIntersect(LineSegment l, QuadSegment c)
   {
     double[] y = new double[3];
     double[] x = new double[3];
@@ -1453,8 +1457,9 @@ public class Area implements Shape, Cloneable
    * Returns the intersections between a line and a cubic segment
    * This is done through combining the line's equation with the
    * parametric form of the Bezier and solving the resulting quadratic.
+   * This is package-private to avoid an accessor method. 
    */
-  private Intersection[] lineCubicIntersect(LineSegment l, CubicSegment c)
+  Intersection[] lineCubicIntersect(LineSegment l, CubicSegment c)
   {
     double[] y = new double[4];
     double[] x = new double[4];
@@ -1560,8 +1565,9 @@ public class Area implements Shape, Cloneable
   /**
    * Returns the intersection between two lines, or null if there is no
    * intersection.
+   * This is package-private to avoid an accessor method.
    */
-  private Intersection linesIntersect(LineSegment a, LineSegment b)
+  Intersection linesIntersect(LineSegment a, LineSegment b)
   {
     Point2D P1 = a.P1;
     Point2D P2 = a.P2;
@@ -1605,8 +1611,9 @@ public class Area implements Shape, Cloneable
   /**
    * Determines if two points are equal, within an error margin
    * 'snap distance'
+   * This is package-private to avoid an accessor method.
    */
-  private boolean pointEquals(Point2D a, Point2D b)
+  boolean pointEquals(Point2D a, Point2D b)
   {
     return (a.equals(b) || a.distance(b) < PE_EPSILON);
   }
index 0dc9ede..70f7de7 100644 (file)
@@ -88,15 +88,18 @@ public final class GeneralPath implements Shape, Cloneable
   /** A big number, but not so big it can't survive a few float operations */
   private static final double BIG_VALUE = java.lang.Double.MAX_VALUE / 10.0;
 
-  /** The winding rule. */
-  private int rule;
+  /** The winding rule.
+   * This is package-private to avoid an accessor method.
+   */
+  int rule;
 
   /**
    * The path type in points. Note that xpoints[index] and ypoints[index] maps
    * to types[index]; the control points of quad and cubic paths map as
    * well but are ignored.
+   * This is package-private to avoid an accessor method.
    */
-  private byte[] types;
+  byte[] types;
 
   /**
    * The list of all points seen. Since you can only append floats, it makes
@@ -104,15 +107,18 @@ public final class GeneralPath implements Shape, Cloneable
    * allow a general path of double precision points.
    * Note: Storing x and y coords seperately makes for a slower transforms,
    * But it speeds up and simplifies box-intersection checking a lot.
+   * These are package-private to avoid accessor methods.
    */
-  private float[] xpoints;
-  private float[] ypoints;
+  float[] xpoints;
+  float[] ypoints;
 
   /** The index of the most recent moveto point, or null. */
   private int subpath = -1;
 
-  /** The next available index into points. */
-  private int index;
+  /** The next available index into points.
+   * This is package-private to avoid an accessor method.
+   */
+  int index;
 
   /**
    * Constructs a GeneralPath with the default (NON_ZERO)
@@ -534,8 +540,9 @@ public final class GeneralPath implements Shape, Cloneable
 
     /**
      * The GeneralPath whose segments are being iterated.
+     * This is package-private to avoid an accessor method.
      */
-    private final GeneralPath path;
+    final GeneralPath path;
 
     /**
      * The affine transformation used to transform coordinates.
index 27809ce..1ae4ed1 100644 (file)
@@ -229,11 +229,11 @@ public class JColorChooser extends JComponent implements Accessible
 
   /**
    * This is a helper method to make the given JDialog block until it is
-   * hidden.
+   * hidden.  This is package-private to avoid an accessor method.
    *
    * @param dialog The JDialog to block.
    */
-  private static void makeModal(JDialog dialog)
+  static void makeModal(JDialog dialog)
   {
     try
       {
index f544aec..863ed20 100644 (file)
@@ -155,7 +155,8 @@ public class JFormattedTextField extends JTextField
   private Object value;
   private int focusLostBehavior = COMMIT_OR_REVERT;
   private AbstractFormatterFactory formatterFactory;
-  private boolean editValid = true;
+  // Package-private to avoid an accessor method.
+  boolean editValid = true;
   
   public JFormattedTextField ()
   {
index c29ba78..8a9d4c8 100644 (file)
@@ -176,12 +176,13 @@ public class RepaintManager
 
   /**
    * The global, shared RepaintManager instance. This is reused for all
-   * components in all windows.
+   * components in all windows.  This is package-private to avoid an accessor
+   * method.
    *
    * @see #currentManager
    * @see #setCurrentManager
    */
-  private static RepaintManager globalManager;
+  static RepaintManager globalManager;
 
   /**
    * Create a new RepaintManager object.
index 0906b8d..d1521af 100644 (file)
@@ -82,6 +82,7 @@ public class Timer implements Serializable
 
   /**
    * DOCUMENT ME!
+   * Package-private to avoid an accessor method.
    */
   private void queueEvent()
   {
@@ -95,8 +96,9 @@ public class Timer implements Serializable
 
   /**
    * DOCUMENT ME!
+   * This is package-private to avoid an accessor method.
    */
-  private void drainEvents()
+  void drainEvents()
   {
     synchronized (queueLock)
       {
index 6245786..667e260 100644 (file)
@@ -466,9 +466,9 @@ public class ToolTipManager extends MouseAdapter implements MouseMotionListener
   /**
    * This method displays the ToolTip. It can figure out the method needed to
    * show it as well (whether to display it in heavyweight/lightweight panel
-   * or a window.)
+   * or a window.)  This is package-private to avoid an accessor method.
    */
-  private void showTip()
+  void showTip()
   {
     if (! enabled || currentComponent == null)
       return;
@@ -535,8 +535,9 @@ public class ToolTipManager extends MouseAdapter implements MouseMotionListener
 
   /**
    * This method hides the ToolTip.
+   * This is package-private to avoid an accessor method.
    */
-  private void hideTip()
+  void hideTip()
   {
     if (currentTip == null || ! currentTip.isVisible() || ! enabled)
       return;
index 9d6fa4e..62acbca 100644 (file)
@@ -119,8 +119,9 @@ public class TransferHandler implements Serializable
   
   /**
    * Clipboard if system clipboard may not be used.
+   * Package-private to avoid an accessor method.
    */
-  private static Clipboard clipboard;
+  static Clipboard clipboard;
   
   private int sourceActions;
   private Icon visualRepresentation;
index c249403..ea18e42 100644 (file)
@@ -1,5 +1,5 @@
 /* DefaultHSBChooserPanel.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -70,38 +70,47 @@ import javax.swing.event.ChangeListener;
  */
 class DefaultHSBChooserPanel extends AbstractColorChooserPanel
 {
-  /** The gradient image displayed. */
-  private transient Image gradientImage;
+  /** The gradient image displayed.
+   * This is package-private to avoid an accessor method.  */
+  transient Image gradientImage;
 
   /** The Panel that holds the gradient image. */
   private transient JPanel gradientPanel;
 
-  /** The track gradient image. */
-  private transient Image trackImage;
+  /** The track gradient image.
+   * This is package-private to avoid an accessor method.  */
+  transient Image trackImage;
 
   /** The panel that holds the track. */
   private transient JPanel trackPanel;
 
-  /** The slider for the locked HSB value. */
-  private transient JSlider slider;
+  /** The slider for the locked HSB value.
+   * This is package-private to avoid an accessor method.  */
+  transient JSlider slider;
 
-  /** The RadioButton that controls the Hue. */
-  private transient JRadioButton hRadio;
+  /** The RadioButton that controls the Hue.
+   * This is package-private to avoid an accessor method.  */
+  transient JRadioButton hRadio;
 
-  /** The RadioButton that controls the Saturation. */
-  private transient JRadioButton sRadio;
+  /** The RadioButton that controls the Saturation.
+   * This is package-private to avoid an accessor method.  */
+  transient JRadioButton sRadio;
 
-  /** The RadioButton that controls the Brightness. */
-  private transient JRadioButton bRadio;
+  /** The RadioButton that controls the Brightness.
+   * This is package-private to avoid an accessor method.  */
+  transient JRadioButton bRadio;
 
-  /** The JSpinner that controls the Hue. */
-  private transient JSpinner hSpinner;
+  /** The JSpinner that controls the Hue.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner hSpinner;
 
-  /** The JSpinner that controls the Saturation. */
-  private transient JSpinner sSpinner;
+  /** The JSpinner that controls the Saturation.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner sSpinner;
 
-  /** The JSpinner that controls the Brightness. */
-  private transient JSpinner bSpinner;
+  /** The JSpinner that controls the Brightness.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner bSpinner;
 
   /** The default width of the gradient image. */
   private static final int imgWidth = 200;
@@ -132,20 +141,26 @@ class DefaultHSBChooserPanel extends AbstractColorChooserPanel
   /** The JLabel that displays the value of Blue. */
   private transient JLabel bFull;
 
-  /** The point that is displayed in the gradient image. */
-  private transient Point gradientPoint = new Point();
+  /** The point that is displayed in the gradient image.
+   * Package-private to avoid an accessor method.
+   */
+  transient Point gradientPoint = new Point();
 
   /**
    * This indicates that the change to the slider or point is triggered
    * internally.
+   * This is package-private to avoid an accessor method.
    */
-  private transient boolean internalChange = false;
+  transient boolean internalChange = false;
 
-  /** This indicates that the change to the spinner is triggered internally. */
-  private transient boolean spinnerTrigger = false;
+  /** This indicates that the change to the spinner is triggered
+   * internally.
+   * This is package-private to avoid an accessor method.  */
+  transient boolean spinnerTrigger = false;
 
-  /** This int identifies which spinner is currently locked. */
-  private transient int locked = -1;
+  /** This int identifies which spinner is currently locked.
+   * This is package-private to avoid an accessor method.  */
+  transient int locked = -1;
 
   /** This value indicates that the Hue spinner is locked. */
   static final int HLOCKED = 0;
@@ -159,8 +174,9 @@ class DefaultHSBChooserPanel extends AbstractColorChooserPanel
   /**
    * This method indicates that the mouse event is in the process of being
    * handled.
+   * This is package-private to avoid an accessor method.
    */
-  private transient boolean handlingMouse;
+  transient boolean handlingMouse;
 
   /**
    * This helper class handles mouse events on the gradient image.
@@ -716,8 +732,9 @@ class DefaultHSBChooserPanel extends AbstractColorChooserPanel
   /**
    * This method calls the appropriate method to update the gradient image
    * depending on which HSB value is constant.
+   * This is package-private to avoid an accessor method.
    */
-  private void updateImage()
+  void updateImage()
   {
     switch (locked)
       {
@@ -750,8 +767,9 @@ class DefaultHSBChooserPanel extends AbstractColorChooserPanel
   /**
    * This method updates the slider in response to making a different HSB
    * property the constant.
+   * This is package-private to avoid an accessor method.
    */
-  private void updateSlider()
+  void updateSlider()
   {
     if (slider == null)
       return;
@@ -779,8 +797,9 @@ class DefaultHSBChooserPanel extends AbstractColorChooserPanel
   /**
    * This method updates the track gradient image depending on which HSB
    * property is constant.
+   * This is package-private to avoid an accessor method.
    */
-  private void updateTrack()
+  void updateTrack()
   {
     switch (locked)
       {
index 9f85164..7cd9bfe 100644 (file)
@@ -109,17 +109,20 @@ class DefaultRGBChooserPanel extends AbstractColorChooserPanel
     }
   }
 
-  /** Whether the color change was initiated by the spinners. */
-  private transient boolean spinnerChange = false;
+  /** Whether the color change was initiated by the spinners.
+   * This is package-private to avoid an accessor method.  */
+  transient boolean spinnerChange = false;
 
-  /** Whether the color change was initiated by the sliders. */
-  private transient boolean sliderChange = false;
+  /** Whether the color change was initiated by the sliders.
+   * This is package-private to avoid an accessor method.  */
+  transient boolean sliderChange = false;
 
   /**
    * Whether the change was forced by the chooser (meaning the color has
    * already been changed).
+   * This is package-private to avoid an accessor method.
    */
-  private transient boolean updateChange = false;
+  transient boolean updateChange = false;
 
   /** The ChangeListener for the sliders. */
   private transient ChangeListener colorChanger;
@@ -127,14 +130,17 @@ class DefaultRGBChooserPanel extends AbstractColorChooserPanel
   /** The ChangeListener for the spinners. */
   private transient ChangeListener spinnerHandler;
 
-  /** The slider that handles the red values. */
-  private transient JSlider R;
+  /** The slider that handles the red values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSlider R;
 
-  /** The slider that handles the green values. */
-  private transient JSlider G;
+  /** The slider that handles the green values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSlider G;
 
-  /** The slider that handles the blue values. */
-  private transient JSlider B;
+  /** The slider that handles the blue values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSlider B;
 
   /** The label for the red slider. */
   private transient JLabel RLabel;
@@ -145,14 +151,17 @@ class DefaultRGBChooserPanel extends AbstractColorChooserPanel
   /** The label for the blue slider. */
   private transient JLabel BLabel;
 
-  /** The spinner that handles the red values. */
-  private transient JSpinner RSpinner;
+  /** The spinner that handles the red values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner RSpinner;
 
-  /** The spinner that handles the green values. */
-  private transient JSpinner GSpinner;
+  /** The spinner that handles the green values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner GSpinner;
 
-  /** The spinner that handles the blue values. */
-  private transient JSpinner BSpinner;
+  /** The spinner that handles the blue values.
+   * This is package-private to avoid an accessor method.  */
+  transient JSpinner BSpinner;
 
   /**
    * Creates a new DefaultRGBChooserPanel object.
index 20a72f4..505de95 100644 (file)
@@ -501,10 +501,11 @@ class DefaultSwatchChooserPanel extends AbstractColorChooserPanel
 
     /**
      * This method adds the given color to the beginning of the swatch panel.
+     * Package-private to avoid an accessor method.
      *
      * @param c The color to add.
      */
-    private void addColorToQueue(Color c)
+    void addColorToQueue(Color c)
     {
       if (--start == -1)
        start = numRows * numCols - 1;
index f876a78..0526961 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicArrowButton.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -80,17 +80,22 @@ public class BasicArrowButton extends JButton implements SwingConstants
   /**
    * The color the arrow is painted in if disabled and the bottom and right
    * edges of the button.
+   * This is package-private to avoid an accessor method.
    */
-  private transient Color shadow = Color.GRAY;
+  transient Color shadow = Color.GRAY;
 
   /**
    * The color the arrow is painted in if enabled and the bottom and right
    * edges of the button.
+   * This is package-private to avoid an accessor method.
    */
-  private transient Color darkShadow = Color.DARK_GRAY;
+  transient Color darkShadow = Color.DARK_GRAY;
 
-  /** The top and left edges of the button. */
-  private transient Color highlight = Color.WHITE;
+  /**
+   * The top and left edges of the button.
+   * This is package-private to avoid an accessor method.
+   */
+  transient Color highlight = Color.WHITE;
 
   /** The border around the ArrowButton. */
   private transient Border buttonBorder = new Border()
index 61af43b..5b87628 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicColorChooserUI.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -141,11 +141,14 @@ public class BasicColorChooserUI extends ColorChooserUI
   /** The PropertyChangeListener for the JColorChooser. */
   protected PropertyChangeListener propertyChangeListener;
 
-  /** The JColorChooser. */
-  private JColorChooser chooser;
+  /**
+   * The JColorChooser.
+   * This is package-private to avoid an accessor method.
+   */
+  JColorChooser chooser;
 
   /** The JTabbedPane that is used. */
-  private JTabbedPane pane;
+  JTabbedPane pane;
 
   /** The Container that holds the preview panel. */
   private Container prevContainer;
@@ -219,10 +222,11 @@ public class BasicColorChooserUI extends ColorChooserUI
   /**
    * This method adds tabs to the JTabbedPane for the chooserPanels defined in
    * the JColorChooser.
+   * This is package-private to avoid an accessor method.
    *
    * @param panels The Panels that need tabs to be made for them.
    */
-  private void makeTabs(AbstractColorChooserPanel[] panels)
+  void makeTabs(AbstractColorChooserPanel[] panels)
   {
     pane.removeAll();
     for (int i = 0; i < panels.length; i++)
@@ -255,10 +259,11 @@ public class BasicColorChooserUI extends ColorChooserUI
   /**
    * This is a helper method that swaps the existing preview panel with the
    * given panel.
+   * This is package-private to avoid an accessor method.
    *
    * @param preview The new preview panel.
    */
-  private void updatePreviewPanel(JComponent preview)
+  void updatePreviewPanel(JComponent preview)
   {
     if (prevContainer == null)
       {
index 9044ed7..2199886 100644 (file)
@@ -160,17 +160,22 @@ public class BasicComboBoxUI extends ComboBoxUI
   private Color highlight;
   private Color lightHighlight;
 
-  /* Size of the largest item in the comboBox */
-  private Dimension largestItemSize;
+  /* Size of the largest item in the comboBox
+   * This is package-private to avoid an accessor method.
+   */
+  Dimension largestItemSize;
 
   // It seems that JComboBox doesn't have a border set explicitely. So we just
   // paint the border everytime combo box is displayed. 
 
-  /* border insets for this JComboBox*/
-  private static final Insets borderInsets = new Insets(2, 2, 2, 2);
+  /* border insets for this JComboBox
+   * This is package-private to avoid an accessor method. */
+  static final Insets borderInsets = new Insets(2, 2, 2, 2);
 
   // Width of the arrow button  
-  private static int arrowButtonWidth = 15;
+  // This is package-private to avoid an accessor method.
+  // FIXME: has wrong name for a constant.
+  static final int arrowButtonWidth = 15;
 
   // FIXME: This fields aren't used anywhere at this moment.
   protected Dimension cachedMinimumSize;
index 4867c27..2a87761 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicDesktopIconUI.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -302,8 +302,9 @@ public class BasicDesktopIconUI extends DesktopIconUI
   /** The MouseListener responsible for reacting to MouseEvents on the JDesktopIcon. */
   private transient MouseInputListener mouseHandler;
 
-  /** The Button in the JDesktopIcon responsible for deiconifying it. */
-  private transient BoundButton button;
+  /** The Button in the JDesktopIcon responsible for deiconifying it.
+   * This is package-private to avoid an accessor method. */
+  transient BoundButton button;
 
   /** The PropertyChangeListener listening to the JDesktopIcon. */
   private transient PropertyChangeListener propertyHandler;
index fd2832a..bc81887 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicInternalFrameTitlePane.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -644,8 +644,9 @@ public class BasicInternalFrameTitlePane extends JComponent
   /**
    * The label used to display the title. This label is not added to the
    * TitlePane.
+   * This is package-private to avoid an accessor method.
    */
-  private transient JLabel title;
+  transient JLabel title;
 
   /**
    * Creates a new BasicInternalFrameTitlePane object that is used in the
index 8cb0cb4..589ec9d 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicListUI.java --
-   Copyright (C) 2002, 2004 Free Software Foundation, Inc.
+   Copyright (C) 2002, 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -491,10 +491,11 @@ public class BasicListUI extends ListUI
   /**
    * Marks the current layout as damaged and requests revalidation from the
    * JList.
+   * This is package-private to avoid an accessor method.
    *
    * @see #updateLayoutStateNeeded
    */
-  private void damageLayout()
+  void damageLayout()
   {
     updateLayoutStateNeeded = 1;
     list.revalidate();
index 95d53b0..f86dd75 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicOptionPaneUI.java --
-   Copyright (C) 2004 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -439,7 +439,8 @@ public class BasicOptionPaneUI extends OptionPaneUI
   protected JOptionPane optionPane;
 
   /** The size of the icons. */
-  private static int iconSize = 36;
+  // FIXME: wrong name for a constant.
+  private static final int iconSize = 36;
 
   /** The foreground color for the message area. */
   private transient Color messageForeground;
@@ -451,22 +452,28 @@ public class BasicOptionPaneUI extends OptionPaneUI
   private transient Border buttonBorder;
 
   /** The string used to describe OK buttons. */
-  private static String OK_STRING = "OK";
+  // FIXME: wrong name for a constant.
+  private static final String OK_STRING = "OK";
 
   /** The string used to describe Yes buttons. */
-  private static String YES_STRING = "Yes";
+  // FIXME: wrong name for a constant.
+  private static final String YES_STRING = "Yes";
 
   /** The string used to describe No buttons. */
-  private static String NO_STRING = "No";
+  // FIXME: wrong name for a constant.
+  private static final String NO_STRING = "No";
 
   /** The string used to describe Cancel buttons. */
-  private static String CANCEL_STRING = "Cancel";
+  // FIXME: wrong name for a constant.
+  private static final String CANCEL_STRING = "Cancel";
 
-  /** The container for the message area. */
-  private transient Container messageAreaContainer;
+  /** The container for the message area.
+   * This is package-private to avoid an accessor method. */
+  transient Container messageAreaContainer;
 
-  /** The container for the buttons. */
-  private transient Container buttonContainer;
+  /** The container for the buttons.
+   * This is package-private to avoid an accessor method.  */
+  transient Container buttonContainer;
 
   /**
    * A helper class that implements Icon. This is used temporarily until
@@ -1227,8 +1234,9 @@ public class BasicOptionPaneUI extends OptionPaneUI
   /**
    * This method resets the value in the inputComponent to the
    * initialSelectionValue property.
+   * This is package-private to avoid an accessor method.
    */
-  private void resetSelectedValue()
+  void resetSelectedValue()
   {
     if (inputComponent != null)
       {
index 796af56..c131c86 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicPopupMenuUI.java
-   Copyright (C) 2002, 2004 Free Software Foundation, Inc.
+   Copyright (C) 2002, 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -73,8 +73,9 @@ public class BasicPopupMenuUI extends PopupMenuUI
   /* PopupMenuListener listens to popup menu events fired by JPopupMenu*/
   private transient PopupMenuListener popupMenuListener;
 
-  /* ComponentListener listening to popupMenu's invoker. */
-  private TopWindowListener topWindowListener;
+  /* ComponentListener listening to popupMenu's invoker.
+   * This is package-private to avoid an accessor method.  */
+  TopWindowListener topWindowListener;
 
   /**
    * Creates a new BasicPopupMenuUI object.
index 7e06067..a7de701 100644 (file)
@@ -645,8 +645,9 @@ public class BasicScrollBarUI extends ScrollBarUI implements LayoutManager,
   /**
    * This method calculates the preferred size since calling
    * getPreferredSize() returns a cached value.
+   * This is package-private to avoid an accessor method.
    */
-  private void calculatePreferredSize()
+  void calculatePreferredSize()
   {
     // System.err.println(this + ".calculatePreferredSize()");
     int height;
@@ -1248,12 +1249,13 @@ public class BasicScrollBarUI extends ScrollBarUI implements LayoutManager,
    * This method returns the value in the scrollbar's range given the y
    * coordinate. If the value is out of range, it will return the closest
    * legal value.
+   * This is package-private to avoid an accessor method.
    *
    * @param yPos The y coordinate to calculate a value for.
    *
    * @return The value for the y coordinate.
    */
-  private int valueForYPosition(int yPos)
+  int valueForYPosition(int yPos)
   {
     int min = scrollbar.getMinimum();
     int max = scrollbar.getMaximum();
@@ -1280,12 +1282,13 @@ public class BasicScrollBarUI extends ScrollBarUI implements LayoutManager,
    * This method returns the value in the scrollbar's range given the x
    * coordinate. If the value is out of range, it will return the closest
    * legal value.
+   * This is package-private to avoid an accessor method.
    *
    * @param xPos The x coordinate to calculate a value for.
    *
    * @return The value for the x coordinate.
    */
-  private int valueForXPosition(int xPos)
+  int valueForXPosition(int xPos)
   {
     int min = scrollbar.getMinimum();
     int max = scrollbar.getMaximum();
index a08acc9..9879d8d 100644 (file)
@@ -2158,12 +2158,13 @@ public class BasicSliderUI extends SliderUI
 
   /**
    * This method finds the closest value that has a tick associated with it.
+   * This is package-private to avoid an accessor method.
    *
    * @param value The value to search from.
    *
    * @return The closest value that has a tick associated with it.
    */
-  private int findClosestTick(int value)
+  int findClosestTick(int value)
   {
     int min = slider.getMinimum();
     int max = slider.getMaximum();
index 987f86b..3b8596e 100644 (file)
@@ -1,5 +1,5 @@
 /* BasicSplitPaneDivider.java --
-   Copyright (C) 2003, 2004 Free Software Foundation, Inc.
+   Copyright (C) 2003, 2004, 2005 Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -157,8 +157,9 @@ public class BasicSplitPaneDivider extends Container
   /**
    * Keeps track of where the divider should be placed when using one touch
    * expand buttons.
+   * This is package-private to avoid an accessor method.
    */
-  private transient int currentDividerLocation = 1;
+  transient int currentDividerLocation = 1;
 
   /** DOCUMENT ME! */
   private transient Border tmpBorder = new Border()
@@ -493,10 +494,11 @@ public class BasicSplitPaneDivider extends Container
    * using one touch expand buttons. Location 0 is the left (or top) most
    * location. Location 1 is the middle. Location 2 is the right (or bottom)
    * most location.
+   * This is package-private to avoid an accessor method.
    *
    * @param locationIndex The location to move to.
    */
-  private void moveDividerTo(int locationIndex)
+  void moveDividerTo(int locationIndex)
   {
     Insets insets = splitPane.getInsets();
     switch (locationIndex)
index c2af60f..aca7049 100644 (file)
@@ -1181,20 +1181,25 @@ public class BasicTabbedPaneUI extends TabbedPaneUI implements SwingConstants
     }
   }
 
-  /** The button that increments the current scroll location. */
-  private transient ScrollingButton incrButton;
+  /** The button that increments the current scroll location.
+   * This is package-private to avoid an accessor method.  */
+  transient ScrollingButton incrButton;
 
-  /** The button that decrements the current scroll location. */
-  private transient ScrollingButton decrButton;
+  /** The button that decrements the current scroll location.
+   * This is package-private to avoid an accessor method.  */
+  transient ScrollingButton decrButton;
 
-  /** The viewport used to display the tabs. */
-  private transient ScrollingViewport viewport;
+  /** The viewport used to display the tabs.
+   * This is package-private to avoid an accessor method.  */
+  transient ScrollingViewport viewport;
 
-  /** The panel inside the viewport that paints the tabs. */
-  private transient ScrollingPanel panel;
+  /** The panel inside the viewport that paints the tabs.
+   * This is package-private to avoid an accessor method.  */
+  transient ScrollingPanel panel;
 
-  /** The starting visible tab in the run in SCROLL_TAB_MODE. */
-  private transient int currentScrollLocation;
+  /** The starting visible tab in the run in SCROLL_TAB_MODE.
+   * This is package-private to avoid an accessor method.  */
+  transient int currentScrollLocation;
 
   /** A reusable rectangle. */
   protected Rectangle calcRect;
@@ -1305,14 +1310,17 @@ public class BasicTabbedPaneUI extends TabbedPaneUI implements SwingConstants
   /** The tab pane that this UI paints. */
   protected JTabbedPane tabPane;
 
-  /** The current layout manager for the tabPane. */
-  private transient LayoutManager layoutManager;
+  /** The current layout manager for the tabPane.
+   * This is package-private to avoid an accessor method.  */
+  transient LayoutManager layoutManager;
 
-  /** The rectangle that describes the tab area's position and size. */
-  private transient Rectangle tabAreaRect;
+  /** The rectangle that describes the tab area's position and size.
+   * This is package-private to avoid an accessor method.  */
+  transient Rectangle tabAreaRect;
 
-  /** The rectangle that describes the content area's position and size. */
-  private transient Rectangle contentRect;
+  /** The rectangle that describes the content area's position and
+   * size.  This is package-private to avoid an accessor method.  */
+  transient Rectangle contentRect;
 
   /**
    * Creates a new BasicTabbedPaneUI object.
@@ -1325,10 +1333,11 @@ public class BasicTabbedPaneUI extends TabbedPaneUI implements SwingConstants
   /**
    * This method creates a ScrollingButton that  points in the appropriate
    * direction for an increasing button.
+   * This is package-private to avoid an accessor method.
    *
    * @return The increase ScrollingButton.
    */
-  private ScrollingButton createIncreaseButton()
+  ScrollingButton createIncreaseButton()
   {
     if (incrButton == null)
       incrButton = new ScrollingButton(SwingConstants.NORTH);
@@ -1343,10 +1352,11 @@ public class BasicTabbedPaneUI extends TabbedPaneUI implements SwingConstants
   /**
    * This method creates a ScrollingButton that points in the appropriate
    * direction for a decreasing button.
+   * This is package-private to avoid an accessor method.
    *
    * @return The decrease ScrollingButton.
    */
-  private ScrollingButton createDecreaseButton()
+  ScrollingButton createDecreaseButton()
   {
     if (decrButton == null)
       decrButton = new ScrollingButton(SwingConstants.SOUTH);
@@ -1361,12 +1371,13 @@ public class BasicTabbedPaneUI extends TabbedPaneUI implements SwingConstants
   /**
    * This method finds the point to set the view  position at given the index
    * of a tab. The tab will be the first visible tab in the run.
+   * This is package-private to avoid an accessor method.
    *
    * @param index The index of the first visible tab.
    *
    * @return The position of the first visible tab.
    */
-  private Point findPointForIndex(int index)
+  Point findPointForIndex(int index)
   {
     int tabPlacement = tabPane.getTabPlacement();
     int selectedIndex = tabPane.getSelectedIndex();
index fad0108..18477c9 100644 (file)
@@ -82,8 +82,9 @@ import javax.swing.plaf.UIResource;
  */
 public class BasicToolBarUI extends ToolBarUI implements SwingConstants
 {
-  /** Static owner of all DragWindows. */
-  private static JFrame owner = new JFrame();
+  /** Static owner of all DragWindows.
+   * This is package-private to avoid an accessor method.  */
+  static JFrame owner = new JFrame();
 
   /** The border used when the JToolBar is in nonrollover mode. */
   private static Border nonRolloverBorder;
@@ -94,8 +95,9 @@ public class BasicToolBarUI extends ToolBarUI implements SwingConstants
   /** The last known BorderLayout constraint before floating. */
   protected String constraintBeforeFloating;
 
-  /** The last known orientation of the JToolBar before floating. */
-  private int lastGoodOrientation;
+  /** The last known orientation of the JToolBar before floating.
+   * This is package-private to avoid an accessor method.  */
+  int lastGoodOrientation;
 
   /** The color of the border when it is dockable. */
   protected Color dockingBorderColor;
@@ -136,20 +138,24 @@ public class BasicToolBarUI extends ToolBarUI implements SwingConstants
    */
   private transient Window floatFrame;
 
-  /** The original parent of the JToolBar. */
-  private transient Container origParent;
+  /** The original parent of the JToolBar.
+   * This is package-private to avoid an accessor method.  */
+  transient Container origParent;
 
-  /** A hashtable of components and their original borders. */
-  private transient Hashtable borders;
+  /** A hashtable of components and their original borders.
+   * This is package-private to avoid an accessor method.  */
+  transient Hashtable borders;
 
   /** A window listener for the floatable frame. */
   private transient WindowListener windowListener;
 
-  /** A set of cached bounds of the JToolBar. */
-  private transient Dimension cachedBounds;
+  /** A set of cached bounds of the JToolBar.
+   * This is package-private to avoid an accessor method.  */
+  transient Dimension cachedBounds;
 
-  /** The cached orientation of the JToolBar. */
-  private transient int cachedOrientation;
+  /** The cached orientation of the JToolBar.
+   * This is package-private to avoid an accessor method.  */
+  transient int cachedOrientation;
 
   /**
    * This method creates a new <code>BasicToolBarUI</code> object for the given JToolBar.
@@ -1086,8 +1092,9 @@ public class BasicToolBarUI extends ToolBarUI implements SwingConstants
 
     /**
      * Creates a new DragWindow object.
+     * This is package-private to avoid an accessor method.
      */
-    private DragWindow()
+    DragWindow()
     {
       super(owner);
     }
index fa0040b..6b464b5 100644 (file)
@@ -883,7 +883,8 @@ public abstract class JTextComponent extends JComponent
     return getUI().getEditorKit(this).getActions();
   }
     
-  private Document doc;
+  // This is package-private to avoid an accessor method.
+  Document doc;
   private Caret caret;
   private Highlighter highlighter;
   private Color caretColor;
index c984bd8..c0e688a 100644 (file)
@@ -56,7 +56,8 @@ import javax.swing.undo.UndoableEdit;
  */
 public final class StringContent implements AbstractDocument.Content, Serializable
 {
-  private char[] content;
+  // This is package-private to avoid an accessor method.
+  char[] content;
 
   private int count;
 
@@ -156,7 +157,8 @@ public final class StringContent implements AbstractDocument.Content, Serializab
       this.offset = offset;
     }
 
-    private void setOffset(int offset)
+    // This is package-private to avoid an accessor method.
+    void setOffset(int offset)
     {
       this.offset = this.offset >= 0 ? offset : -1;
     }
@@ -290,7 +292,8 @@ public final class StringContent implements AbstractDocument.Content, Serializab
       }
   }
 
-  private void checkLocation(int where, int len) throws BadLocationException
+  // This is package-private to avoid an accessor method.
+  void checkLocation(int where, int len) throws BadLocationException
   {
     if (where < 0)
       throw new BadLocationException("Invalid location", 1);
index 8f7d213..f210d4f 100644 (file)
@@ -144,19 +144,22 @@ public class DocumentParser
   /**
    * This field is used to access the identically named
    * methods of the outer class.
+   * This is package-private to avoid an accessor method.
    */
-  private DocumentParser parser = this;
+  DocumentParser parser = this;
 
   /**
    * The callback.
+   * This is package-private to avoid an accessor method.
    */
-  private HTMLEditorKit.ParserCallback callBack;
+  HTMLEditorKit.ParserCallback callBack;
 
   /**
    * The reference to the working class of HTML parser that is
    * actually used to parse the document.
+   * This is package-private to avoid an accessor method.
    */
-  private gnuParser gnu;
+  gnuParser gnu;
 
   /**
    * Creates a new parser that uses the given DTD to access data on the
index 7981706..30f77ca 100644 (file)
@@ -126,14 +126,16 @@ public class ParserDelegator
 
   /**
    * The callback.
+   * This is package-private to avoid an accessor method.
    */
-  private HTMLEditorKit.ParserCallback callBack;
+  HTMLEditorKit.ParserCallback callBack;
 
   /**
    * The reference to the working class of HTML parser that is
    * actually used to parse the document.
+   * This is package-private to avoid an accessor method.
    */
-  private gnuParser gnu;
+  gnuParser gnu;
 
   /**
    * Parses the HTML document, calling methods of the provided