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

Unified Diff: courgette/disassembler_win32.cc

Issue 2583373002: [Courgette] Simple AssemblyProgram and Disassembler cleanups. (Closed)
Patch Set: Tune up header inclusion. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « courgette/disassembler_win32.h ('k') | courgette/program_detector.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: courgette/disassembler_win32.cc
diff --git a/courgette/disassembler_win32.cc b/courgette/disassembler_win32.cc
index c2824397e6780b3127546c588c9905a93926b80e..e48b0f190453f506dd6e3e9d5c3e75501975312b 100644
--- a/courgette/disassembler_win32.cc
+++ b/courgette/disassembler_win32.cc
@@ -4,9 +4,6 @@
#include "courgette/disassembler_win32.h"
-#include <stddef.h>
-#include <stdint.h>
-
#include <algorithm>
#include "base/bind.h"
@@ -224,26 +221,24 @@ bool DisassemblerWin32::ParseHeader() {
return Good();
}
-bool DisassemblerWin32::Disassemble(AssemblyProgram* target) {
+bool DisassemblerWin32::Disassemble(AssemblyProgram* program) {
if (!ok())
return false;
- target->set_image_base(image_base());
-
if (!ParseAbs32Relocs())
return false;
ParseRel32RelocsFromSections();
- PrecomputeLabels(target);
- RemoveUnusedRel32Locations(target);
+ PrecomputeLabels(program);
+ RemoveUnusedRel32Locations(program);
- if (!target->GenerateInstructions(
+ if (!program->GenerateInstructions(
base::Bind(&DisassemblerWin32::ParseFile, base::Unretained(this)))) {
return false;
}
- target->DefaultAssignIndexes();
+ program->DefaultAssignIndexes();
return true;
}
« no previous file with comments | « courgette/disassembler_win32.h ('k') | courgette/program_detector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698