RFR: JDK-8313643: Update HarfBuzz to 8.2.2 [v2]

[Phil Race]

On Wed, 25 Oct 2023 17:14:27 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>> Any specific reason why we need to remove `range-loop-analysis` flag as part of this update? Were there any build issues?
>
> @jayathirthrao
> 
> Removed `range-loop-analysis` as part of cleaning up the disabled warning section. I see that the following flag was added as part of https://github.com/openjdk/jdk/pull/993 to disable warnings for clang on Xcode 12.
> Details here: https://github.com/openjdk/jdk/pull/993#issuecomment-722382063.
> 
> I suppose we need to add it back then?

It depends. Perhaps the code that triggered the warning is no longer the same ?
And we just moved to xcode 14 didn't we ? So xcode 12 is now 2 versions back.
What you could try is to take a repo WITHOUT the new version of harfbuzz (ie one without your PR),
remove the disabling of this warning and see if xcode 14 build it - if it does, then the issue is resolved by
the new xcode and we may still need it for the new change. If the problem is still there with xcode 14 and the old (ie current existing) code, then its resolved by the hb update and we definitely don't need it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16322#discussion_r1372120649