Hi Phil,
thanks a lot for the review. Please see my comments inline:
On Wed, Oct 16, 2013 at 12:48 AM, Phil Race wrote:
> Hello Volker,
>
> I just remembered & got back to these changes - my email inbox isn't a very
> good reminder system.
>
> I am OK with everything I see in the client areas ex
Hello Volker,
I just remembered & got back to these changes - my email inbox isn't a
very good reminder system.
I am OK with everything I see in the client areas except as below :-
/* Very basic start for AIX - feel free to complete ..
169 */
We should remove this comment. If that means
Sent: Wednesday, September 18, 2013 3:23 PM
To: Volker Simonis
Cc: serviceability-...@openjdk.java.net; 2d-...@openjdk.java.net;
ppc-aix-port-...@openjdk.java.net; Java Core Libs; awt-...@openjdk.java.net
Subject: Re: [OpenJDK 2D-Dev] RFR(L): 8024854: Basic changes and files to build
the class library o
Hi Phil,
I'm open to any good ideas.
The current solution is pragmatic, simple and it has no impact on
existing platforms.
The other possibility I see is to put the 'dladdr' implantation into
its own library (something equivalent to "libjli") and link that
library to "libawt". But I currently do
Hi,
w.r.t the one issue below : is the awt load library code the only place you
need/are doing this ? If someone else (hotspot, core-libs) already
assented to this
then I guess I could too but I'd like to hear for sure that hotspot and
core-libs
teams both agree to this approach and whether th
Hi Phil,
thank you for looking at the changes. Please find my answers inline:
On Thu, Sep 19, 2013 at 12:22 AM, Phil Race wrote:
> Volker,
>
> I've skimmed the client related changes - mostly based on the descriptions.
> I'll look in more detail as soon as I can after JavaOne. We also have som
Volker,
I've skimmed the client related changes - mostly based on the descriptions.
I'll look in more detail as soon as I can after JavaOne. We also have some
engineers out on vacation who might want to look at these too.
A couple of things that stood out :
"On AIX, fontconfig is not a standard