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

[Pavel Tisnovsky]

Andrew Hughes wrote:
> ----- Original Message -----
>> ----- Original Message -----
>>> 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.
>>>
>> You say this is based on "OpenJDK7 sources".  What fix is this from
>> and
>> why is there no bug reference in this patch?
> 
> This appears to be a fragment of:

yes, it is

> 
> changeset:   3068:1d4340015b85
> user:        srl
> date:        Mon Dec 06 16:10:01 2010 -0800
> summary:     6886358: layout code update
> 
> Why not apply the whole fix?  Including chunks like this makes things
> untraceable and harder to upstream.

well this fix is 880 kilobytes long and it changes a lot of code. I'm not fully
against to use the whole fix, but I'm a bit scared if it does introduce
some new bugs into a now-stable IcedTea6.

> 
>>> 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
>>>
>>>
>>> [Text Documents:coverage-table.patch]
>>>
>> --
>> Andrew :)
>>
>> Free Java Software Engineer
>> Red Hat, Inc. (http://www.redhat.com)
>>
>> PGP Key: 248BDC07 (https://keys.indymedia.org/)
>> Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07
>>
>>
>