[OpenJDK 2D-Dev] [PATCH] SurfaceManagerFactory

[Dmitri Trembovetski]

   OK, I've submitted the bug (6675596) and applied the
   fix, and in the process of testing it.

   I can tell that someone didn't build on windows
   since the build fails there =) The fix should have deleted
   src/windows/classes/sun/java2d/SurfaceManagerFactory.java
   just like it did the solaris/ one.

   In general, if the fix touches shared code it is
   very advisable to build on all platforms - at
   least on 32-bit solaris/linux/windows.

   Thanks,
     Dmitri


Dmitri Trembovetski wrote:
> 
>   Roman, I'm sorry, I'm completely swamped with a high
>   priority bug. I'll try to file a bug and a code review
>   tomorrow.
> 
>   Dmitri
> 
> 
> Roman Kennke wrote:
>> Hi,
>>
>>>> This makes sense, because BufImgVolatileSurfaceManager is required for
>>>> all targets.
>>>    Yep, you're right, that should cover it.
>>>
>>>    I'd wait for another reviewer and if there are no
>>>    objections we could integrate it.
>>
>> Any progress on this one?
>>
>> Cheers, Roman
>>