+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
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);
/* 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.
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();
/**
* The attributes of the current HTML element.
+ * Package-private to avoid an accessor method.
*/
- private htmlAttributeSet attributes =
+ htmlAttributeSet attributes =
htmlAttributeSet.EMPTY_HTML_ATTRIBUTE_SET;
/**
/**
* 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();
* A hooks for operations, preceeding call to handleStartTag().
* The method is called when the HTML opening tag ((like <table>)
* 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);
/**
* @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;
/**
* @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;
/**
* @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.
/**
* @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.
*
* @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;
/*************************************************************************/
/**
* @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.
/**
* @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
/**
* @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.
/*******************************/
// 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();
return -1;
}
- private synchronized Rectangle
+ synchronized Rectangle
getCharacterBounds(int i)
{
TextComponentPeer tcp = (TextComponentPeer)getPeer();
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.
*/
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.
/**
* 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
* 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();
* 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];
* 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];
/**
* 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;
/**
* 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);
}
/** 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
* 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)
/**
* 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.
/**
* 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
{
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 ()
{
/**
* 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.
/**
* DOCUMENT ME!
+ * Package-private to avoid an accessor method.
*/
private void queueEvent()
{
/**
* DOCUMENT ME!
+ * This is package-private to avoid an accessor method.
*/
- private void drainEvents()
+ void drainEvents()
{
synchronized (queueLock)
{
/**
* 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;
/**
* 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;
/**
* 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;
/* DefaultHSBChooserPanel.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
*/
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;
/** 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;
/**
* 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.
/**
* 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)
{
/**
* 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;
/**
* 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)
{
}
}
- /** 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;
/** 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;
/** 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.
/**
* 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;
/* BasicArrowButton.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
/**
* 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()
/* BasicColorChooserUI.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
/** 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;
/**
* 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++)
/**
* 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)
{
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;
/* BasicDesktopIconUI.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
/** 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;
/* BasicInternalFrameTitlePane.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
/**
* 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
/* 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.
/**
* 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();
/* BasicOptionPaneUI.java --
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2005 Free Software Foundation, Inc.
This file is part of GNU Classpath.
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;
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
/**
* 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)
{
/* 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.
/* 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.
/**
* 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;
* 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();
* 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();
/**
* 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();
/* 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.
/**
* 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()
* 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)
}
}
- /** 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;
/** 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.
/**
* 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);
/**
* 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);
/**
* 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();
*/
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;
/** 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;
*/
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.
/**
* Creates a new DragWindow object.
+ * This is package-private to avoid an accessor method.
*/
- private DragWindow()
+ DragWindow()
{
super(owner);
}
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;
*/
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;
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;
}
}
}
- 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);
/**
* 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
/**
* 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
projects / gcc.git / commitdiff