<i18n dev> RFR: 8285306: Fix typos in java.desktop [v8]

Phil Race prr at openjdk.org
Tue Sep 13 22:05:46 UTC 2022


On Tue, 13 Sep 2022 21:32:44 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> I see now that Phil cryptically said:
> 
> > Regarding changes in gif + freetype
> > diff --git a/src/java.desktop/share/native/libawt/awt/image/gif/gifdecoder.c b/src/java.desktop/share/native/libawt/awt/image/gif/gifdecoder.c 
> > 
> > diff --git a/src/java.desktop/share/native/libfontmanager/freetypeScaler.c b/src/java.desktop/share/native/libfontmanager/freetypeScaler.c 
> > 
> > Please exclude ALL 3rd party libraries from this PR.
> 
> That might be interpreted as stating that `gifdecoder.c` is 3rd party source code (although that was by no means clear to me the first time I read it). I'll revert the changes in that file, and also `src/java.desktop/share/native/libfontmanager/freetypeScaler.c`.

I don't know why I mentioned those two files like that but those particular two are JDK code so are fair game.

I did write 
"Please exclude ALL 3rd party libraries from this PR."
and later :
"We need to revert the native code changes to "libfreetype".
and those points are correct.

You did fix the latter, but there are still some 3rd party files in there that are edited : glext.h, wsutils.h, multiVis.c

-------------

PR: https://git.openjdk.org/jdk/pull/8328


More information about the i18n-dev mailing list