[OpenJDK 2D-Dev] Still waiting for review

[Igor Nekrestyanov]

Hi Mark and Martin,

sorry for delay with our responses.

Many of us were involved into FX prerelease activities and had to 
postpone some of openjdk work.

My understanding of current situation with Martin's fixes is the following:
  - for 2 of Martin's fixes internal formal review procedure is in progress
    (we do review all changes and save approved webrevs and this is 
another reason why we ask for webrevs for contributions)
    Unfortunately this procedure is not public and therefore it is hard 
to track it status. However, it should not take long.
  - one of the suggested fixes actually introduces incompatibility in 
the behavior
    and such change has to be reviewed by CCC committee (this is how we 
call it internally,
     not sure what will be public name when it become public process)
     [My understanding of incompatibility is: metadata written by 
"fixed" jdk will not be parsed normally by older jdk]
   
Andrew Brygin (who actually works on these 3 submissions) will provide 
more accurate details about these 3 bugs later today.

I am not sure what is the current status of Mark's patches but i'll ask.

Sorry again and thank you for your contribution to the project,

-igor

Mark Wielaard wrote:
> Hi,
>
> Same issue here as Martin. The following two patches are still waiting:
> http://article.gmane.org/gmane.comp.java.openjdk.2d.devel/537
> http://article.gmane.org/gmane.comp.java.openjdk.2d.devel/498
>
>   
>> Im feeling pretty frustrated by this. After all, I'm donating the work I
>> invested in fixing these issues, writing test cases, and even reviewing
>> that latest changeset for #6541476. The way things are going so far, I
>> won't likely do that agin, but instead simply work around issues and
>> leave the jdk buggy as it is. You risk losing contributions.
>>     
>
> Martin, if you like we can use IcedTea http://icedtea.classpath.org/ as
> staging area for these fixes. That is what I do after testing. The above
> two fixes are already in IcedTea6 for a couple of months since I know
> other users are seeing the same bugs. That way things are at least
> picked up in a timely manner for the various GNU/Linux distros out
> there. http://icedtea.classpath.org/hg/icedtea6
>
> Cheers,
>
> Mark
>
>