Bug 79575

Summary: Fix unused variable warnings in HarfBuzzShaperBase
Product: WebKit Reporter: Adrienne Walker <enne>
Component: New BugsAssignee: Adrienne Walker <enne>
Status: RESOLVED FIXED    
Severity: Normal CC: bashi, enne
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Use ASSERT_UNUSED kling: review+

Description Adrienne Walker 2012-02-25 10:42:46 PST 
Fix unused variable warnings in HarfBuzzShaperBase
Comment 1 Adrienne Walker 2012-02-25 10:47:21 PST 
This was failing on ChromiumOS builds: http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28tegra2%29/builds/4524/steps/cbuildbot/logs/stdio

--SNIP--
third_party/WebKit/Source/WebCore/platform/graphics/harfbuzz/HarfBuzzShaperBase.cpp: In function 'void WebCore::normalizeSpacesAndMirrorChars(const UChar*, UChar*, int, WebCore::HarfBuzzShaperBase::NormalizeMode)':
chromeos-chrome-19.0.1053.0_alpha-r1: third_party/WebKit/Source/WebCore/platform/graphics/harfbuzz/HarfBuzzShaperBase.cpp:58:10: error: variable 'error' set but not used [-Werror=unused-but-set-variable]
chromeos-chrome-19.0.1053.0_alpha-r1: cc1plus: all warnings being treated as errors
--SNIP--
Comment 2 Adrienne Walker 2012-02-25 10:55:03 PST 
Created attachment 128874 [details]
Patch
Comment 3 Adrienne Walker 2012-02-25 11:07:44 PST 
Created attachment 128875 [details]
Use ASSERT_UNUSED
Comment 4 Adrienne Walker 2012-02-25 11:11:41 PST 
Committed r108903: <http://trac.webkit.org/changeset/108903>