DBus font installs [incomplete]
Deepak Bhole
dbhole at redhat.com
Fri Apr 15 07:31:08 PDT 2011
* Dr Andrew John Hughes <ahughes at redhat.com> [2011-04-14 18:48]:
> On 15:32 Thu 14 Apr , Deepak Bhole wrote:
> > > + } else {
> > > +diff -r 9d90acd9f99f src/solaris/native/sun/awt/fontpath.c
> > > +--- openjdk/jdk/src/solaris/native/sun/awt/fontpath.c Wed Feb 16 09:39:35 2011 -0500
> > > ++++ openjdk/jdk/src/solaris/native/sun/awt/fontpath.c Wed Mar 02 09:24:59 2011 -0500
> > > +@@ -517,11 +517,11 @@
> > > + }
> > > +
> > > + JNIEXPORT jstring JNICALL Java_sun_font_FontManager_getFontPath
> > > +-(JNIEnv *env, jclass obj, jboolean noType1) {
> > > ++(JNIEnv *env, jclass obj, jboolean noType1, jboolean recheck) {
> >
> > We should find some other way to recheck (whatever it is doing).
> > Changing a public api, even for a sun.* class is a no-no.
> >
>
> For sun.*, it's possible. I'd prefer it to some convoluted workaround.
>
Wouldn't that make OpenJDK less compatible with (admittedly bad) code
that relies on sun.* API being a certain way?
Deepak
More information about the distro-pkg-dev
mailing list