Name: jk109818 Date: 01/24/2002


FULL PRODUCT VERSION :
java version "1.4.0-beta3"
Java(TM) 2 Runtime Environment, Standard Edition (build
1.4.0-beta3-b84)
Java HotSpot(TM) Client VM (build 1.4.0-beta3-b84, mixed
mode)

FULL OPERATING SYSTEM VERSION :
Microsoft Windows 2000 [Version 5.00.2195]



A DESCRIPTION OF THE PROBLEM :
The documentation for loadChildren in WrappedPlainView says
subclasses can reimplement the method to initialize child
view in a different manner. While this is true, the method
updateChildren in WrappedPlainView is a package method and
uses WrappedLine to update the children. WrappedLine is
also package level so can't be overriden. This prevents
extending the class and replacing the children with
something other than a WrappedLine.

What I really wanted was a WrappableView (so I could turn
wrapping on and off) and the ability to determine the
logical and physical lines created by it and it's children.
While I can extend WrappedPlainView and override
calculateBreakPosition to accomplish the former task, I
can't accomplish the latter without replacing the
inaccessible WrappedLine class with something accessible.



STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
1. Extend WrappedPlainView and override loadChildren.
2. In the overriden loadChildren method load something
other than a WrappedLine.
3. Load some lines of text and dump the view structure.

If the subclass is outside of javax.swing.text the first
two lines will be the new child type and the rest will be
WrappedLines.

EXPECTED VERSUS ACTUAL BEHAVIOR :
I think updateChildren and WrappedLine should be protected.
So should half the other package methods in Views. The
SegmentCache and the package methods in Utilities should be
public so people don't have to rewrite everything to use
Views. Otherwise these classes are useless for anything
other than simple example code.

ERROR MESSAGES/STACK TRACES THAT OCCUR :
StateInvariantError(nothing changed).

This bug can be reproduced always.

CUSTOMER WORKAROUND :
Don't extend WrappedPlainView.
(Review ID: 137725)
======================================================================

Name: rmT116609 Date: 09/24/2002


 DESCRIPTION OF THE PROBLEM :
Please change all package private and private fields and methods in these classes to protected
1. javax.swing.text.WrappedPlainView(cannot subclass because of this)
2. javax.swing.text.Utilities
3. javax.swing.text.SegmentCache
4. javax.swing.text.StateInvariantError
Please make all classes public also!!!! .

I CANNOT SUBCLASS WrappedPlainView and override the methods I want because of the flaws above. I would need to copy WrappedPlainView(which when I do does not work, because there are more classes that are package private that I have to copy)


(Review ID: 164895)
======================================================================