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

Issue 311001: Extract constants from Breakpad.h. (Closed)

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

Description

 Extract constants from Breakpad.h.

 This is done to allow ios implementation to use the same constants.

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Total comments: 2

Patch Set 4 : '' #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M src/client/mac/Breakpad.xcodeproj/project.pbxproj View 1 2 3 4 chunks +4 lines, -0 lines 0 comments Download
M src/client/mac/Framework/Breakpad.h View 1 2 3 2 chunks +5 lines, -47 lines 1 comment Download
M src/client/mac/Framework/Breakpad.mm View 1 2 3 1 chunk +8 lines, -9 lines 0 comments Download
A src/client/mac/Framework/BreakpadDefines.h View 3 1 chunk +72 lines, -0 lines 0 comments Download
M src/client/mac/crash_generation/ConfigFile.mm View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M src/client/mac/sender/crash_report_sender.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M src/client/mac/sender/crash_report_sender.m View 1 2 3 1 chunk +6 lines, -6 lines 0 comments Download
M src/client/mac/sender/uploader.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M src/client/mac/sender/uploader.m View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5
qsr
13 years ago #1
Mark Mentovai
http://breakpad.appspot.com/311001/diff/3010/4004 File src/client/mac/Framework/BreakpadDefines.h (right): http://breakpad.appspot.com/311001/diff/3010/4004#newcode76 Line 76: #define BREAKPAD_BOOTSTRAP_PARENT_PORT "com.Breakpad.BootstrapParent" Put this back where you ...
13 years ago #2
qsr
http://breakpad.appspot.com/311001/diff/3010/4004 File src/client/mac/Framework/BreakpadDefines.h (right): http://breakpad.appspot.com/311001/diff/3010/4004#newcode76 Line 76: #define BREAKPAD_BOOTSTRAP_PARENT_PORT "com.Breakpad.BootstrapParent" On 2011/10/11 14:14:32, Mark Mentovai ...
13 years ago #3
qsr
PTAL
13 years ago #4
Mark Mentovai
13 years ago #5
LGTM

http://breakpad.appspot.com/311001/diff/5002/6003
File src/client/mac/Framework/Breakpad.h (right):

http://breakpad.appspot.com/311001/diff/5002/6003#newcode46
Line 46: #define BREAKPAD_BOOTSTRAP_PARENT_PORT   
"com.Breakpad.BootstrapParent"
My only thought here is that if this doesn’t make sense on iOS, not defining it
in a header that might be included on iOS keeps you from accidentally building
code that actually does try to use this on that platform.
Sign in to reply to this message.

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