RFR: JDK-8301998: Update HarfBuzz to 7.0.1 [v2]

Harshitha Onkar honkar at openjdk.org
Fri Mar 3 20:52:07 UTC 2023


On Fri, 3 Mar 2023 20:30:23 GMT, Phil Race <prr at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Updating.txt changes
>
> src/java.desktop/share/native/libharfbuzz/hb-icu.h line 1:
> 
>> 1: /*
> 
> We don't need/use hb-icu.h and hb-icu.cc, so why are they here ?
> They were there in harfbuzz source for a long time and we've never needed them before and
> given what their purpose is, I don't see why we need them now.
> Are you sure you followed the process of adding in ONLY new files which were needed so that we successfully
> compiler ?
> Please double check ALL new files you added are really needed.

@prrace hb-icu.h was referenced in hb-unicode.cc (existing file), but I missed to see that it is included only when certain flags are set. Thank you for catching it.

I will double check all the newly added files again.

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

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



More information about the client-libs-dev mailing list