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

Issue 1562273002: Use range-based for loops in linux/minidump_writer/minidump_writer.cc. (Closed)

Created:
4 years, 11 months ago by Lei Zhang
Modified:
4 years, 11 months ago
Reviewers:
Mark Mentovai
CC:
google-breakpad-dev_googlegroups.com
Base URL:
https://chromium.googlesource.com/breakpad/breakpad.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Use range-based for loops in linux/minidump_writer/minidump_writer.cc. Also fix lint errors. R=mark@chromium.org Committed: https://chromium.googlesource.com/breakpad/breakpad/+/41440eaa17ec3173365fd0b1e7196500e8c55238

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -45 lines) Patch
M src/client/linux/minidump_writer/minidump_writer.cc View 12 chunks +32 lines, -45 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
Lei Zhang
4 years, 11 months ago (2016-01-07 01:53:33 UTC) #3
Mark Mentovai
LGTM
4 years, 11 months ago (2016-01-07 04:02:12 UTC) #4
Lei Zhang
Committed patchset #1 (id:20001) manually as 41440eaa17ec3173365fd0b1e7196500e8c55238 (presubmit successful).
4 years, 11 months ago (2016-01-07 08:53:12 UTC) #6
bryan.kroger
On 2016/01/07 08:53:12, Lei Zhang wrote: > Committed patchset #1 (id:20001) manually as > 41440eaa17ec3173365fd0b1e7196500e8c55238 ...
4 years, 11 months ago (2016-01-07 19:21:08 UTC) #7
Lei Zhang
On 2016/01/07 19:21:08, bryan.kroger wrote: > On 2016/01/07 08:53:12, Lei Zhang wrote: > > Committed ...
4 years, 11 months ago (2016-01-07 21:28:15 UTC) #8
Lei Zhang
4 years, 11 months ago (2016-01-08 20:04:21 UTC) #9
Message was sent while issue was closed.
On 2016/01/07 21:28:15, Lei Zhang wrote:
> On 2016/01/07 19:21:08, bryan.kroger wrote:
> > On 2016/01/07 08:53:12, Lei Zhang wrote:
> > > Committed patchset #1 (id:20001) manually as
> > > 41440eaa17ec3173365fd0b1e7196500e8c55238 (presubmit successful).
> > 
> > This patch seems to be causing an issue:
> > 
> > src/client/linux/minidump_writer/minidump_writer.cc:837:36: error:
range-based
> > 'for' loops are not allowed in C++98 mode
> >          for (CpuInfoEntry& entry : cpu_info_table) {
> 
> Yes, I need to fiddle with autoconf / automake.

If you sync to commit ae0fdf6 or newer, it should work, assuming ./configure
works.

Powered by Google App Engine
This is Rietveld 408576698