Skip to content

deps/v8: add missing #include "unicode/normlzr.h"

Rodrigo Muino Tomonari requested to merge github/fork/ArchangeGabriel/patch-1 into v6.x-staging

  • The following function from <unicode/normlzr.h> is used: normalize()

  • Until ICU 59, <unicode/normlzr.h> is indirectly included, but this changed with the 59 release. Adding this header has been the right thing to do for many years, so it is backwards compatible and fix compilation with recent ICU.

Refs: https://github.com/nodejs/node/issues/13022

P.S.: This is my first PR in Node, and my first “real” PR in any project, so I don’t expect it to be alright from the start.

Merge request reports