[kkeshava]

1. On Ubuntu 26.04, libre2-dev pulls in libabsl-dev (Abseil 20260107.0), so find_package(absl) in CMakeLists.txt uses the system Abseil. In this Abseil version, the pointer constructor absl::MutexLock(absl::Mutex*) is marked [[deprecated("Use the constructor that takes a reference instead")]]. Because CMakeLists.txt sets:

if (NOT WIN32)
add_definitions ("-Wall -Werror")
endif ()

compiling PhoneNumberOfflineGeocoder::~PhoneNumberOfflineGeocoder (line 80) and PhoneNumberOfflineGeocoder::GetAreaDescription (line 198) fails on absl::MutexLock l(&mu_); with:

error: ‘absl::debian7::MutexLock::MutexLock(absl::debian7::Mutex*)’ is deprecated: Use the constructor that takes a reference instead [-Werror=deprecated-declarations]

Fix for Build C++: In CMakeLists.txt, add -Wno-deprecated-declarations (the same flag used to build locally in opensource_metadata):

if (NOT WIN32)
add_definitions ("-Wall -Werror -Wno-deprecated-declarations")
endif ()

(Note: Using -Wno-deprecated-declarations in CMakeLists.txt is safer than changing absl::MutexLock l(&mu_); to absl::MutexLock l(mu_); in phonenumber_offline_geocoder.cc, because when absl is not installed on the system, CMakeLists.txt falls back to fetching Abseil 273292d from 2021, which only has the pointer constructor MutexLock(Mutex).)*

2. In cpp/test/phonenumbers/phonenumberutil_test.cc, line 1636 already passes the escaped UTF-8 bytes "(650) 253-0000\xE2\x80\x8F", while the comment on line 1637 contained a raw, unescaped U+200F character. So removing the raw U+200F comment.