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

Issue 2728653003: [Courgette] Disassembler code movement and format fix. (Closed)

Created:
3 years, 9 months ago by huangs
Modified:
3 years, 9 months ago
Reviewers:
chrisha, Will Harris
CC:
chromium-reviews, wfh+watch_chromium.org, huangs+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Courgette] Disassembler code movement and format fix. We wish to rename ParseAbs32Relocs() and ParseRel32RelocsFromSections(), and make them part of the Disassembler interface. This requires moving chunks of code. This CL does the code movement as an independent change to reduce noise later. Also including reformatting changes from "git cl format". BUG=660980 Review-Url: https://codereview.chromium.org/2728653003 Cr-Commit-Position: refs/heads/master@{#454068} Committed: https://chromium.googlesource.com/chromium/src/+/dc779d9f0fab468f70ef7fb2cf5029e2373473dc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+126 lines, -127 lines) Patch
M courgette/disassembler_elf_32.h View 2 chunks +5 lines, -5 lines 0 comments Download
M courgette/disassembler_elf_32.cc View 3 chunks +64 lines, -66 lines 0 comments Download
M courgette/disassembler_win32.h View 2 chunks +3 lines, -2 lines 0 comments Download
M courgette/disassembler_win32.cc View 2 chunks +54 lines, -54 lines 0 comments Download

Messages

Total messages: 17 (10 generated)
huangs
Rather mechanical CL of code movement. PTAL.
3 years, 9 months ago (2017-03-01 19:24:03 UTC) #4
Will Harris
lgtm
3 years, 9 months ago (2017-03-01 22:01:54 UTC) #9
huangs
Thanks for chiming in. Committing!
3 years, 9 months ago (2017-03-01 22:15:43 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2728653003/1
3 years, 9 months ago (2017-03-01 22:17:00 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/dc779d9f0fab468f70ef7fb2cf5029e2373473dc
3 years, 9 months ago (2017-03-01 22:23:21 UTC) #15
chrisha
late to the party post-commit lgtm :)
3 years, 9 months ago (2017-03-02 20:01:06 UTC) #16
Will Harris
3 years, 9 months ago (2017-03-02 20:43:03 UTC) #17
Message was sent while issue was closed.
eventually consistent lgtm is best lgtm

Powered by Google App Engine
This is Rietveld 408576698