Chromium Code Reviews

Issue 1418001: Bump breakpad version to version 557, which improves how invalid argument CRT... (Closed)

Created:
10 years, 9 months ago by Sigurður Ásgeirsson
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai, Lei Zhang
CC:
chromium-reviews
Visibility:
Public.

Description

Bump breakpad version to version 557, which improves how invalid argument CRT reports are generated to allow sorting those dumps by stack signature. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43081

Patch Set 1 #

Patch Set 2 : Bump breakpad version to version 557, which improves how invalid argument CRT... #

Unified diffs Side-by-side diffs Stats (+5 lines, -1 line)
M DEPS View 1 chunk +1 line, -1 line 0 comments
M breakpad/breakpad.gyp View 3 chunks +4 lines, -0 lines 0 comments

Messages

Total messages: 7 (0 generated)
Sigurður Ásgeirsson
10 years, 9 months ago (2010-03-26 16:57:53 UTC) #1
Mark Mentovai
That’s quite a large version swing. Watch for falling rocks. LGTM. siggi wrote: > Reviewers: ...
10 years, 9 months ago (2010-03-26 17:01:22 UTC) #2
Sigurður Ásgeirsson
Please take another look, some rock avoidance in there now.
10 years, 8 months ago (2010-03-30 14:14:18 UTC) #3
Mark Mentovai
LGTM
10 years, 8 months ago (2010-03-30 14:18:51 UTC) #4
Mark Mentovai
dynamic_cast crept into breakpad/src/common/dwarf/dwarf2reader.cc. All of our builds use -fno-rtti. The Mac official build builds ...
10 years, 8 months ago (2010-03-30 16:26:52 UTC) #5
Mark Mentovai
I wrote: > dynamic_cast crept into breakpad/src/common/dwarf/dwarf2reader.cc. > All of our builds use -fno-rtti. The ...
10 years, 8 months ago (2010-03-30 16:34:20 UTC) #6
Lei Zhang
10 years, 8 months ago (2010-03-30 18:55:07 UTC) #7
On 2010/03/30 16:34:20, Mark Mentovai wrote:
> I wrote:
> > dynamic_cast crept into breakpad/src/common/dwarf/dwarf2reader.cc.
> > All of our builds use -fno-rtti.  The Mac official build builds this
> > file.
> 
> http://code.google.com/p/google-breakpad/issues/detail?id=379
> 

I think r461 breaks several utility programs on the Linux side. We need to clean
that up too.

Powered by Google App Engine