Reviewer needed - fix for PR1018: JVM fails due to SEGV during rendering some Unicode characters

[Pavel Tisnovsky]

Jiri Vanek wrote:
> On 05/30/2012 03:08 PM, Pavel Tisnovsky wrote:
>> Jiri Vanek wrote:
>>> On 05/30/2012 02:41 PM, Pavel Tisnovsky wrote:
>>>> Hi,
>>>>
>>>> I've created a fix for PR1018: JVM fails due to SEGV during rendering
>>>> some Unicode characters.
>>>> This fix is based on OpenJDK7 sources and it's been successfully
>>>> tested against IcedTea6-head,
>>>> IcedTea6-1.11 and IcedTea6-1.10.
>>>>
>>>> Can anybody please review this fix?
>>>>
>>>> Here's ChangeLog entry:
>>>>
>>>> 2012-05-30  Pavel Tisnovsky<ptisnovs at redhat.com>
>>>>
>>>>           * patches/coverage-table.patch:
>>>>           PR1018: JVM fails due to SEGV during rendering some Unicode
>>>> characters
>>>>           * Makefile.am: Updated.
>>>>           * NEWS: Mention bugfix.
>>>>
>>>> (if this fix will be approved, I would like to backport it to 6-1.10
>>>> and 6-1.11 too - the
>>>> patch is the same for all branches)
>>>>
>>>> Thank you in advance,
>>>> Pavel
>>>
>>> I have walked across "6886358: layout code update" fix on openjdk and it
>>> looks correct. But it looks like you have not send complete hg diff (but
>>> only patch of file) with also news and makefile. Can you please repost?
>>>
>>
>> Doh! You are right, I forgot to add hg.diff to the attachments. Should
>> be fixed now ;)
>>
>> Thanks,
>> Pavel
>>
>>> Tahnx, J.
>>
> 
> Thanx. go on and push:) Can you also attach the reproducer I have seen?
> So also the rest of community can have fun with it?
TY!

One reproducer is already stored as an attachment in the IT bugzilla:
http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1018

Here's simpler version I'd like to add in the form of JTreg tests
into IT6/IT7:

import javax.swing.*;

public class bug1018 {
    public static void main(String[] args) {
        JFrame frame = new JFrame();
        frame.add(new JButton("***\u0b03\u0b03\u0b03***"));
        frame.pack();
        frame.dispose();
    }
}

> 
> J.