Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(15)

Issue 331001: Add API on iOS to generate dump on demand. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 11 months ago by qsr
Modified:
12 years, 11 months ago
Reviewers:
Mark Mentovai
CC:
google-breakpad-dev_googlegroups.com
Base URL:
http://google-breakpad.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add API on iOS to generate dump on demand.

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M src/client/ios/Breakpad.h View 2 chunks +11 lines, -0 lines 0 comments Download
M src/client/ios/Breakpad.mm View 1 5 chunks +55 lines, -2 lines 0 comments Download

Messages

Total messages: 3
qsr
12 years, 11 months ago #1
Mark Mentovai
LGTM http://breakpad.appspot.com/331001/diff/1/3 File src/client/ios/Breakpad.mm (right): http://breakpad.appspot.com/331001/diff/1/3#newcode476 Line 476: dumpId.c_str()); This would all fit on one ...
12 years, 11 months ago #2
qsr
12 years, 11 months ago #3
http://breakpad.appspot.com/331001/diff/1/3
File src/client/ios/Breakpad.mm (right):

http://breakpad.appspot.com/331001/diff/1/3#newcode476
Line 476: dumpId.c_str());
On 2011/12/08 17:56:50, Mark Mentovai wrote:
> This would all fit on one line.

Done.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1004:630ec63f810e-tainted