From 512d32eeb34acde09bb97322bc6868b19db41bd1 Mon Sep 17 00:00:00 2001 From: tromey Date: Tue, 26 Apr 2005 18:57:40 +0000 Subject: * 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. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@98786 138bc75d-0d04-0410-961f-82ee72b054a4 --- libjava/javax/swing/JColorChooser.java | 4 +- libjava/javax/swing/JFormattedTextField.java | 3 +- libjava/javax/swing/RepaintManager.java | 5 +- libjava/javax/swing/Timer.java | 4 +- libjava/javax/swing/ToolTipManager.java | 7 +- libjava/javax/swing/TransferHandler.java | 3 +- .../swing/colorchooser/DefaultHSBChooserPanel.java | 79 ++++++++++++++-------- .../swing/colorchooser/DefaultRGBChooserPanel.java | 43 +++++++----- .../colorchooser/DefaultSwatchChooserPanel.java | 3 +- .../javax/swing/plaf/basic/BasicArrowButton.java | 15 ++-- .../swing/plaf/basic/BasicColorChooserUI.java | 17 +++-- .../javax/swing/plaf/basic/BasicComboBoxUI.java | 15 ++-- .../javax/swing/plaf/basic/BasicDesktopIconUI.java | 7 +- .../plaf/basic/BasicInternalFrameTitlePane.java | 5 +- libjava/javax/swing/plaf/basic/BasicListUI.java | 5 +- .../javax/swing/plaf/basic/BasicOptionPaneUI.java | 30 +++++--- .../javax/swing/plaf/basic/BasicPopupMenuUI.java | 7 +- .../javax/swing/plaf/basic/BasicScrollBarUI.java | 9 ++- libjava/javax/swing/plaf/basic/BasicSliderUI.java | 3 +- .../swing/plaf/basic/BasicSplitPaneDivider.java | 8 ++- .../javax/swing/plaf/basic/BasicTabbedPaneUI.java | 49 ++++++++------ libjava/javax/swing/plaf/basic/BasicToolBarUI.java | 33 +++++---- libjava/javax/swing/text/JTextComponent.java | 3 +- libjava/javax/swing/text/StringContent.java | 9 ++- .../swing/text/html/parser/DocumentParser.java | 9 ++- .../swing/text/html/parser/ParserDelegator.java | 6 +- 26 files changed, 238 insertions(+), 143 deletions(-) (limited to 'libjava/javax') diff --git a/libjava/javax/swing/JColorChooser.java b/libjava/javax/swing/JColorChooser.java index 27809cee09e..1ae4ed195f2 100644 --- a/libjava/javax/swing/JColorChooser.java +++ b/libjava/javax/swing/JColorChooser.java @@ -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 { diff --git a/libjava/javax/swing/JFormattedTextField.java b/libjava/javax/swing/JFormattedTextField.java index f544aec117f..863ed204a1e 100644 --- a/libjava/javax/swing/JFormattedTextField.java +++ b/libjava/javax/swing/JFormattedTextField.java @@ -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 () { diff --git a/libjava/javax/swing/RepaintManager.java b/libjava/javax/swing/RepaintManager.java index c29ba787e37..8a9d4c8e8cf 100644 --- a/libjava/javax/swing/RepaintManager.java +++ b/libjava/javax/swing/RepaintManager.java @@ -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. diff --git a/libjava/javax/swing/Timer.java b/libjava/javax/swing/Timer.java index 0906b8d93bb..d1521af1058 100644 --- a/libjava/javax/swing/Timer.java +++ b/libjava/javax/swing/Timer.java @@ -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) { diff --git a/libjava/javax/swing/ToolTipManager.java b/libjava/javax/swing/ToolTipManager.java index 6245786afa3..667e26094d4 100644 --- a/libjava/javax/swing/ToolTipManager.java +++ b/libjava/javax/swing/ToolTipManager.java @@ -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; diff --git a/libjava/javax/swing/TransferHandler.java b/libjava/javax/swing/TransferHandler.java index 9d6fa4e3f7c..62acbca5897 100644 --- a/libjava/javax/swing/TransferHandler.java +++ b/libjava/javax/swing/TransferHandler.java @@ -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; diff --git a/libjava/javax/swing/colorchooser/DefaultHSBChooserPanel.java b/libjava/javax/swing/colorchooser/DefaultHSBChooserPanel.java index c2494039b8d..ea18e4296e4 100644 --- a/libjava/javax/swing/colorchooser/DefaultHSBChooserPanel.java +++ b/libjava/javax/swing/colorchooser/DefaultHSBChooserPanel.java @@ -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) { diff --git a/libjava/javax/swing/colorchooser/DefaultRGBChooserPanel.java b/libjava/javax/swing/colorchooser/DefaultRGBChooserPanel.java index 9f85164041b..7cd9bfe3a93 100644 --- a/libjava/javax/swing/colorchooser/DefaultRGBChooserPanel.java +++ b/libjava/javax/swing/colorchooser/DefaultRGBChooserPanel.java @@ -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. diff --git a/libjava/javax/swing/colorchooser/DefaultSwatchChooserPanel.java b/libjava/javax/swing/colorchooser/DefaultSwatchChooserPanel.java index 20a72f438a8..505de95dce6 100644 --- a/libjava/javax/swing/colorchooser/DefaultSwatchChooserPanel.java +++ b/libjava/javax/swing/colorchooser/DefaultSwatchChooserPanel.java @@ -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; diff --git a/libjava/javax/swing/plaf/basic/BasicArrowButton.java b/libjava/javax/swing/plaf/basic/BasicArrowButton.java index f876a783fe2..0526961a9c9 100644 --- a/libjava/javax/swing/plaf/basic/BasicArrowButton.java +++ b/libjava/javax/swing/plaf/basic/BasicArrowButton.java @@ -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() diff --git a/libjava/javax/swing/plaf/basic/BasicColorChooserUI.java b/libjava/javax/swing/plaf/basic/BasicColorChooserUI.java index 61af43bbf70..5b876284c8b 100644 --- a/libjava/javax/swing/plaf/basic/BasicColorChooserUI.java +++ b/libjava/javax/swing/plaf/basic/BasicColorChooserUI.java @@ -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) { diff --git a/libjava/javax/swing/plaf/basic/BasicComboBoxUI.java b/libjava/javax/swing/plaf/basic/BasicComboBoxUI.java index 9044ed76c86..21998868b0e 100644 --- a/libjava/javax/swing/plaf/basic/BasicComboBoxUI.java +++ b/libjava/javax/swing/plaf/basic/BasicComboBoxUI.java @@ -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; diff --git a/libjava/javax/swing/plaf/basic/BasicDesktopIconUI.java b/libjava/javax/swing/plaf/basic/BasicDesktopIconUI.java index 4867c27d8ca..2a87761eea7 100644 --- a/libjava/javax/swing/plaf/basic/BasicDesktopIconUI.java +++ b/libjava/javax/swing/plaf/basic/BasicDesktopIconUI.java @@ -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; diff --git a/libjava/javax/swing/plaf/basic/BasicInternalFrameTitlePane.java b/libjava/javax/swing/plaf/basic/BasicInternalFrameTitlePane.java index fd2832a4073..bc81887d7ee 100644 --- a/libjava/javax/swing/plaf/basic/BasicInternalFrameTitlePane.java +++ b/libjava/javax/swing/plaf/basic/BasicInternalFrameTitlePane.java @@ -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 diff --git a/libjava/javax/swing/plaf/basic/BasicListUI.java b/libjava/javax/swing/plaf/basic/BasicListUI.java index 8cb0cb4c68c..589ec9d4374 100644 --- a/libjava/javax/swing/plaf/basic/BasicListUI.java +++ b/libjava/javax/swing/plaf/basic/BasicListUI.java @@ -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(); diff --git a/libjava/javax/swing/plaf/basic/BasicOptionPaneUI.java b/libjava/javax/swing/plaf/basic/BasicOptionPaneUI.java index 95d53b00bd2..f86dd75e634 100644 --- a/libjava/javax/swing/plaf/basic/BasicOptionPaneUI.java +++ b/libjava/javax/swing/plaf/basic/BasicOptionPaneUI.java @@ -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) { diff --git a/libjava/javax/swing/plaf/basic/BasicPopupMenuUI.java b/libjava/javax/swing/plaf/basic/BasicPopupMenuUI.java index 796af5643cc..c131c86b1dd 100644 --- a/libjava/javax/swing/plaf/basic/BasicPopupMenuUI.java +++ b/libjava/javax/swing/plaf/basic/BasicPopupMenuUI.java @@ -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. diff --git a/libjava/javax/swing/plaf/basic/BasicScrollBarUI.java b/libjava/javax/swing/plaf/basic/BasicScrollBarUI.java index 7e06067fa49..a7de70101dc 100644 --- a/libjava/javax/swing/plaf/basic/BasicScrollBarUI.java +++ b/libjava/javax/swing/plaf/basic/BasicScrollBarUI.java @@ -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(); diff --git a/libjava/javax/swing/plaf/basic/BasicSliderUI.java b/libjava/javax/swing/plaf/basic/BasicSliderUI.java index a08acc9bdad..9879d8ddb96 100644 --- a/libjava/javax/swing/plaf/basic/BasicSliderUI.java +++ b/libjava/javax/swing/plaf/basic/BasicSliderUI.java @@ -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(); diff --git a/libjava/javax/swing/plaf/basic/BasicSplitPaneDivider.java b/libjava/javax/swing/plaf/basic/BasicSplitPaneDivider.java index 987f86bdb1b..3b8596e7e2d 100644 --- a/libjava/javax/swing/plaf/basic/BasicSplitPaneDivider.java +++ b/libjava/javax/swing/plaf/basic/BasicSplitPaneDivider.java @@ -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) diff --git a/libjava/javax/swing/plaf/basic/BasicTabbedPaneUI.java b/libjava/javax/swing/plaf/basic/BasicTabbedPaneUI.java index c2af60f1186..aca70498715 100644 --- a/libjava/javax/swing/plaf/basic/BasicTabbedPaneUI.java +++ b/libjava/javax/swing/plaf/basic/BasicTabbedPaneUI.java @@ -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(); diff --git a/libjava/javax/swing/plaf/basic/BasicToolBarUI.java b/libjava/javax/swing/plaf/basic/BasicToolBarUI.java index fad0108e096..18477c9dd75 100644 --- a/libjava/javax/swing/plaf/basic/BasicToolBarUI.java +++ b/libjava/javax/swing/plaf/basic/BasicToolBarUI.java @@ -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 BasicToolBarUI 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); } diff --git a/libjava/javax/swing/text/JTextComponent.java b/libjava/javax/swing/text/JTextComponent.java index fa0040bfd44..6b464b5def0 100644 --- a/libjava/javax/swing/text/JTextComponent.java +++ b/libjava/javax/swing/text/JTextComponent.java @@ -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; diff --git a/libjava/javax/swing/text/StringContent.java b/libjava/javax/swing/text/StringContent.java index c984bd892ee..c0e688a8dcf 100644 --- a/libjava/javax/swing/text/StringContent.java +++ b/libjava/javax/swing/text/StringContent.java @@ -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); diff --git a/libjava/javax/swing/text/html/parser/DocumentParser.java b/libjava/javax/swing/text/html/parser/DocumentParser.java index 8f7d213b36a..f210d4f8804 100644 --- a/libjava/javax/swing/text/html/parser/DocumentParser.java +++ b/libjava/javax/swing/text/html/parser/DocumentParser.java @@ -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 diff --git a/libjava/javax/swing/text/html/parser/ParserDelegator.java b/libjava/javax/swing/text/html/parser/ParserDelegator.java index 798170630b7..30f77ca7c67 100644 --- a/libjava/javax/swing/text/html/parser/ParserDelegator.java +++ b/libjava/javax/swing/text/html/parser/ParserDelegator.java @@ -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 -- cgit v1.2.1