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

Issue 1571293003: Define intptr and uintptr in a more generic way (Closed)

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

Description

Define intptr and uintptr in a more generic way MSVC does not have the __PTRDIFF_TYPE__ macro defined, so I use the standard [u]intptr_t types instead. Compilation tested on windows, linux and mac. BUG= R=mark@chromium.org Committed: https://chromium.googlesource.com/breakpad/breakpad/+/860d7f5da1895c47bd07aecfff0d907ab177ad4b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -6 lines) Patch
M src/common/dwarf/types.h View 1 chunk +2 lines, -6 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
labath
4 years, 11 months ago (2016-01-11 13:19:56 UTC) #2
Mark Mentovai
LGTM
4 years, 11 months ago (2016-01-11 17:37:47 UTC) #3
Mark Mentovai
4 years, 11 months ago (2016-01-11 17:38:34 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
860d7f5da1895c47bd07aecfff0d907ab177ad4b (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698