summaryrefslogtreecommitdiffstats
path: root/third_party/harfbuzz-ng
diff options
context:
space:
mode:
authorthakis <thakis@chromium.org>2015-12-31 09:26:06 -0800
committerCommit bot <commit-bot@chromium.org>2015-12-31 17:26:58 +0000
commit29de2c811de5e5fa6eb238bbcd45796907c037bd (patch)
treeac1ea9902b9958b3af28eda50a4301bf56c3ecf5 /third_party/harfbuzz-ng
parent92469c8c4b7da052273dee1a31308663d1dd1e3d (diff)
downloadchromium_src-29de2c811de5e5fa6eb238bbcd45796907c037bd.zip
chromium_src-29de2c811de5e5fa6eb238bbcd45796907c037bd.tar.gz
chromium_src-29de2c811de5e5fa6eb238bbcd45796907c037bd.tar.bz2
Prepare for -Wall for third-party code, -Wextra for chromium_code
This lands patch set 16 of https://codereview.chromium.org/1551753002/ (reviewed there), except for the changes to build/common.gypi and build/config/compiler/BUILD.gn. The changes to these two files are likely to break some bot I don't know about, and I don't want to keep reverting and relanding all these files. So this lands the parts that are hopefully harmless separately. BUG=573250 TBR=thestig Review URL: https://codereview.chromium.org/1557733002 Cr-Commit-Position: refs/heads/master@{#367252}
Diffstat (limited to 'third_party/harfbuzz-ng')
-rw-r--r--third_party/harfbuzz-ng/README.chromium4
-rw-r--r--third_party/harfbuzz-ng/src/hb-coretext.cc5
2 files changed, 4 insertions, 5 deletions
diff --git a/third_party/harfbuzz-ng/README.chromium b/third_party/harfbuzz-ng/README.chromium
index 1484e11..107d359 100644
--- a/third_party/harfbuzz-ng/README.chromium
+++ b/third_party/harfbuzz-ng/README.chromium
@@ -18,3 +18,7 @@ tarball's src folder over to src, then checking for removed or added files
and update our build recipes in BUILD.gn and harfbuzz.gyp accordingly, update
the NEWS file from HarfBuzz' release notes, and bump the version numbers
in README.chromium.
+
+Local changes:
+* merge https://github.com/behdad/harfbuzz/pull/203
+* merge https://github.com/behdad/harfbuzz/pull/206
diff --git a/third_party/harfbuzz-ng/src/hb-coretext.cc b/third_party/harfbuzz-ng/src/hb-coretext.cc
index 13ba5d9..316d281 100644
--- a/third_party/harfbuzz-ng/src/hb-coretext.cc
+++ b/third_party/harfbuzz-ng/src/hb-coretext.cc
@@ -693,7 +693,6 @@ resize_and_retry:
scratch += old_scratch_used;
scratch_size -= old_scratch_used;
}
-retry:
{
string_ref = CFStringCreateWithCharactersNoCopy (NULL,
pchars, chars_len,
@@ -1129,10 +1128,6 @@ fail:
* AAT shaper
*/
-HB_SHAPER_DATA_ENSURE_DECLARE(coretext_aat, face)
-HB_SHAPER_DATA_ENSURE_DECLARE(coretext_aat, font)
-
-
/*
* shaper face data
*/