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

Anthony Petrov anthony.petrov at oracle.com
Fri May 24 10:25:51 UTC 2013

Thank you. Swing team will review your fix.

-- best regards, Anthony

On 05/24/2013 02:21 PM, dmitry markov wrote:

Hi Anthony,

I have just fixed bug title. Thank you, Dmitry On 24/05/2013 14:08, Vladislav Karnaukhov wrote: Anthony, Artem,

thank you for the clarification. Dmitry, could you please edit bug title and re-submit your request? Regards, - Vlad On 5/24/2013 01:51 PM, Anthony Petrov wrote: Hi Vladislav,

I'm not sure about an enforced policy. However, please read this: http://mail.openjdk.java.net/pipermail/jdk8-dev/2013-April/002414.html Pushing changesets with all-caps titles may result in them being rolled-back and black-listed. You may save your (and other engineers') time by providing human-readable synopses to bugs that you're going to fix in OpenJDK. -- best regards, Anthony On 05/24/2013 01:29 PM, Vladislav Karnaukhov wrote: Hello Anthony,

do we have a policy with requires us to provide bug description in lowercase-only? I can see that may other bugs were reviewed and pushed without their titles edited... I'm not aware of such a policy; could you please point at it? Regards, - Vlad On 5/24/2013 01:23 PM, Anthony Petrov wrote: Dmitry,

Please fix the case of the letters in the synopsis for this bug in JBS, and also please remember to use the fixed title in your hg commit message. You shouldn't push a changeset with all-caps title in the repository. Thanks. -- best regards, Anthony On 05/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. Thanks, Dmitry.

More information about the swing-dev mailing list