Please review fix - approved

[Phil Race]

I see what you mean. I'm not sure how the raw file was created or last 
edited but that
is actually the original shared file, and its like that in the 
"raw_files" from Clemen's
last webrev generated on 20th April. So the new solaris one is really a
just "cp'd" to the solaris location and I then edited the shared one which
does have a final newline.
So it does work, I guess a missing final newline messes up at least my
version of webrev .. which is the shared one on the internal devtools 
server.

But I'll give it a trailing new line anyway.

-phil.

On 6/18/2010 12:58 AM, Andrew Brygin wrote:
> Hi Phil,
>
> fix looks fine to me.
>
> BTW, it looks like that 'New' representation for 
> src/solaris/classes/sun/java2d/pisces/META-INF/services/sun.java2d.pipe.RenderingEngine 
>
> misses the last line, that looks a bit misleading. Corresponding raw 
> file and patch look fine though.
>
> Thanks,
> Andrew
>
> Phil Race wrote:
>> Clemens and Andrew,
>>
>> http://cr.openjdk.java.net/~prr/6961633/
>>
>> has the fix for being unable to load the JulesRenderingEngine
>> as service in openjdk on windows.
>> The fix is simply to revert the shared file and make a new
>> solaris/linux file which references Jules.
>>
>> Without this fix openjdk is dead on Windows.
>>
>> -phil.
>