commit | bd461369eaf05528c47b7865073466f9d97a549d | [log] [tgz] |
---|---|---|
author | mukesh agrawal <quiche@chromium.org> | Fri Nov 08 17:34:22 2013 -0800 |
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Tue Nov 19 06:07:30 2013 +0000 |
tree | 83d6f5ddcb2902299c0a2b47a34fef6cd9a5c78f | |
parent | 31ce564d64f68172299d4cde60edd7a5c004b38e [diff] |
shill: re-enable -Werror During the migration to platform2, we lost the |Werror| flag to the c/c++ compiler. Add that back, to help keep the code clean. While there: move some struct timeval initializations to value initialization. This is less verbose, and avoids missing field initializer warnings. (Or, it would avoid the warnings if we weren't disabling them for other reasons.) BUG=chromium:312543 TEST=build for peppy and daisy_spring Build commands -------------- USE="asan clang" FEATURES="test" P2_TEST_FILTER="shill::*" emerge-peppy platform2 FEATURES="test" P2_TEST_FILTER="shill::*" emerge-daisy_spring platform2 Change-Id: I6cb91cf5eb17625987a4dcf78c7a493df8ab1572 Reviewed-on: https://chromium-review.googlesource.com/176297 Tested-by: mukesh agrawal <quiche@chromium.org> Reviewed-by: Paul Stewart <pstew@chromium.org> Commit-Queue: mukesh agrawal <quiche@chromium.org>