[8] Review request for 8014863 Line break calculations in Java 7 are incorrect. (original) (raw)

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri May 24 11:35:38 UTC 2013

On 5/24/2013 11:46 AM, dmitry markov wrote:

Hello,

Could toy review the fix: bug: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=8014863 webrev: http://cr.openjdk.java.net/~vkarnauk/8014863/webrev.00/ The fix removes break points caching from GlyphView. If some text is inserted/removed into/from the document contained several elements, all elements views will re-calculate their break points.

For example the ParagraphView.calculateMinorAxisRequirements method

can invoke several methods like getBreakWeight() and calculateMinorAxisRequirements() that both leads to GlyphView.getBreakSpot() method invocation on the same method when the text is not changed. So the breakSpots cache is still useful in this case. May be there are some missed places during text updating where the breakSpots cache should be cleaned.

The getNumberOfTextLines() method in the test uses the JEditorPane on the main thread. There is the invokeOnEDT() method in test Util class that helps to return a result from the EDT thread.

Thanks, Alexandr.

Thanks, Dmitry.

More information about the swing-dev mailing list