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

Issue 211023: GDB wrapper refactoring for memcheck_analyze.py... (Closed)

Created:
11 years, 3 months ago by Timur Iskhodzhanov
Modified:
9 years, 6 months ago
CC:
chromium-reviews_googlegroups.com, Nirnimesh, Timur Iskhodzhanov, stuartmorgan, pam+watch_chromium.org
Visibility:
Public.

Description

GDB wrapper refactoring for memcheck_analyze.py (This changelist is prepared by Alexander Potapenko, cc'ed) This is a small refactoring that moves the GDB querying logic into a separate module to allow us using it from tsan_analyze as well (we need it finally). The diff is almost equal to http://codereview.chromium.org/175026, I've only swapped two lines to make it applicable to the current trunk version. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26911

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+90 lines, -76 lines) Patch
A tools/valgrind/gdb_helper.py View 1 1 chunk +87 lines, -0 lines 0 comments Download
M tools/valgrind/memcheck_analyze.py View 1 2 3 chunks +3 lines, -76 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Timur Iskhodzhanov
Hi Dan, Can you please review this changelist? Thank you, Timur Iskhodzhanov
11 years, 3 months ago (2009-09-18 13:56:21 UTC) #1
dank
LGTM but fix the copyright date on the new file, should be 2009.
11 years, 3 months ago (2009-09-18 16:06:02 UTC) #2
not_the_right_glider
On 2009/09/18 16:06:02, dank wrote: > LGTM but fix the copyright date on the new ...
11 years, 3 months ago (2009-09-23 10:22:46 UTC) #3
dank
On 2009/09/23 10:22:46, ramosian.glider wrote: > FYI: gdb cannot find the debug info for the ...
11 years, 3 months ago (2009-09-23 10:31:12 UTC) #4
not_the_right_glider
On 2009/09/23 10:31:12, dank wrote: > On 2009/09/23 10:22:46, ramosian.glider wrote: > > FYI: gdb ...
11 years, 3 months ago (2009-09-23 10:34:26 UTC) #5
dank
11 years, 3 months ago (2009-09-23 10:38:36 UTC) #6
On 2009/09/23 10:34:26, ramosian.glider wrote:
> Well, then we don't have the line numbers for the Memcheck bot on Mac OS,
> because it's in the Release mode. Is that intentional?

Nope.  I think that one needs to be switched to Debug, too,
not sure how it escaped.

Powered by Google App Engine
This is Rietveld 408576698