diff options
author | dcheng <dcheng@chromium.org> | 2015-07-06 12:01:07 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-06 19:02:43 +0000 |
commit | b1d375b8f86dc380baad68d0c7ffe84c01c6591b (patch) | |
tree | 5fd39193ceaaca20297df5104bf96a106e7d086d /components/update_client | |
parent | 031ce3b33ef0970136c157a35605d8c2917d58e8 (diff) | |
download | chromium_src-b1d375b8f86dc380baad68d0c7ffe84c01c6591b.zip chromium_src-b1d375b8f86dc380baad68d0c7ffe84c01c6591b.tar.gz chromium_src-b1d375b8f86dc380baad68d0c7ffe84c01c6591b.tar.bz2 |
Fix some clang warnings with -Wmissing-braces in components.
Clang warns if there are missing braces around a subobject
initializer. The most common idiom that triggers this is:
STRUCT s = {0};
if the first field of STRUCT is itself a struct. This can
be more simply written as:
STRUCT s = {};
which also prevents the warning from firing.
BUG=505297
Review URL: https://codereview.chromium.org/1210153013
Cr-Commit-Position: refs/heads/master@{#337433}
Diffstat (limited to 'components/update_client')
-rw-r--r-- | components/update_client/background_downloader_win.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/components/update_client/background_downloader_win.cc b/components/update_client/background_downloader_win.cc index 7ea2d43..870a299 100644 --- a/components/update_client/background_downloader_win.cc +++ b/components/update_client/background_downloader_win.cc @@ -211,7 +211,7 @@ HRESULT GetJobByteCount(IBackgroundCopyJob* job, if (!job) return E_FAIL; - BG_JOB_PROGRESS job_progress = {0}; + BG_JOB_PROGRESS job_progress = {}; HRESULT hr = job->GetProgress(&job_progress); if (FAILED(hr)) return hr; @@ -295,7 +295,7 @@ struct JobCreationOlderThanDays bool JobCreationOlderThanDays::operator()(IBackgroundCopyJob* job, int num_days) const { - BG_JOB_TIMES times = {0}; + BG_JOB_TIMES times = {}; HRESULT hr = job->GetTimes(×); if (FAILED(hr)) return false; |