Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Issue 6300001: Clang: enable -Wbool-conversions and -Wunused-variables on Linux. (Closed)

Created:
9 years, 11 months ago by hans
Modified:
9 years, 5 months ago
Reviewers:
Nico, Evan Martin
CC:
chromium-reviews, vrk (LEFT CHROMIUM), ncarter (slow), sjl, Alpha Left Google, idana, Raghu Simha, Erik does not do reviews, Paweł Hajdan Jr., cbentzel+watch_chromium.org, ddorwin+watch_chromium.org, Aaron Boodman, acolwell GONE FROM CHROMIUM, annacc, pam+watch_chromium.org, awong, brettw-cc_chromium.org, darin-cc_chromium.org, scherkus (not reviewing), amit, tim (not reviewing), ceee-reviews_chromium.org
Visibility:
Public.

Description

Clang: enable -Wbool-conversions and -Wunused-variables on Linux. -Wbool-conversion warns about EXPECT_EQ(false, blah), so replace that with EXPECT_FALSE(blah). Do the same with EXPECT_EQ(true, blah) for good measure (even though that doesn't generate warnings). Also remove the one instance of an unused variable. BUG=69421 TEST=buildbots all compile and all tests pass Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71431

Patch Set 1 #

Patch Set 2 : Formatting fix #

Total comments: 6

Patch Set 3 : Rebase, undo indent #

Unified diffs Side-by-side diffs Delta from patch set Stats (+178 lines, -194 lines) Patch
M base/allocator/allocator_unittests.cc View 4 chunks +8 lines, -8 lines 0 comments Download
M base/message_loop_unittest.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M base/message_pump_glib_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M base/pr_time_unittest.cc View 1 9 chunks +30 lines, -30 lines 0 comments Download
M build/common.gypi View 1 2 1 chunk +0 lines, -5 lines 0 comments Download
M ceee/ie/broker/cookie_api_module_unittest.cc View 6 chunks +8 lines, -16 lines 0 comments Download
M chrome/browser/extensions/extension_prefs_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/password_manager/password_store_mac_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/syncable/directory_backing_store_unittest.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_contents/navigation_controller_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/gtk/options/languages_page_gtk_unittest.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/tabs/dock_info_unittest.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/resource_dispatcher_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/spellchecker/spellcheck_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome_frame/test/urlmon_moniker_integration_test.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M courgette/encoded_program_fuzz_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M courgette/streams_unittest.cc View 1 2 7 chunks +18 lines, -18 lines 0 comments Download
M courgette/third_party/paged_array_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/audio/audio_util_unittest.cc View 9 chunks +9 lines, -9 lines 0 comments Download
M media/omx/omx_codec_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M net/base/cert_database_nss_unittest.cc View 1 2 12 chunks +16 lines, -17 lines 0 comments Download
M net/base/ssl_config_service_win_unittest.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M net/websockets/websocket_job_unittest.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M o3d/utils/cross/file_text_reader_test.cc View 9 chunks +25 lines, -25 lines 0 comments Download
M o3d/utils/cross/string_reader_test.cc View 9 chunks +19 lines, -19 lines 0 comments Download
M views/view_unittest.cc View 1 2 1 chunk +4 lines, -4 lines 0 comments Download
M webkit/glue/cpp_variant_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin_impl_unittest.cc View 13 chunks +13 lines, -13 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
hans
9 years, 11 months ago (2011-01-13 16:20:08 UTC) #1
Nico
9 years, 11 months ago (2011-01-13 16:44:30 UTC) #2
LG

I was wondering why this touches shared files, since the mac bot should've
caught these already. The explanation is that in most of the shared files, only
the TRUE case was present, which isn't an error. And we don't build o3d on mac.
Sot it's all good.

http://codereview.chromium.org/6300001/diff/2001/base/message_loop_unittest.cc
File base/message_loop_unittest.cc (right):

http://codereview.chromium.org/6300001/diff/2001/base/message_loop_unittest.c...
base/message_loop_unittest.cc:1558:
EXPECT_FALSE(loop.high_resolution_timers_enabled());
This is in an OS_WIN block…I assume you just mass-replaced all over the project?

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
File net/base/cert_database_nss_unittest.cc (right):

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
net/base/cert_database_nss_unittest.cc:326: certs, CertDatabase::TRUSTED_SSL |
CertDatabase::TRUSTED_EMAIL,
revert indentation change

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
net/base/cert_database_nss_unittest.cc:341: certs, CertDatabase::TRUSTED_SSL |
CertDatabase::TRUSTED_EMAIL,
here too

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
net/base/cert_database_nss_unittest.cc:385: certs, CertDatabase::TRUSTED_SSL |
CertDatabase::TRUSTED_EMAIL,
here too

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
net/base/cert_database_nss_unittest.cc:409: certs, CertDatabase::TRUSTED_SSL |
CertDatabase::TRUSTED_EMAIL |
here too

http://codereview.chromium.org/6300001/diff/2001/net/base/cert_database_nss_u...
net/base/cert_database_nss_unittest.cc:485: EXPECT_TRUE(cert_db_.SetCertTrust(
here too

Powered by Google App Engine
This is Rietveld 408576698