From 6ec40350feecd12fe123b19a5ead7740f1c21bac Mon Sep 17 00:00:00 2001 From: Brian Norris Date: Mon, 21 May 2012 10:55:28 -0700 Subject: [PATCH] snapshot: move local defines after #includes Prevents accidental dependencies, where a header *might* use PAGESIZE, e.g. --- snapshot-interface.cc | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/snapshot-interface.cc b/snapshot-interface.cc index 10bd809..9d6ad7f 100644 --- a/snapshot-interface.cc +++ b/snapshot-interface.cc @@ -1,9 +1,3 @@ -#define MYBINARYNAME "model" -#define MYLIBRARYNAME "libmodel.so" -#define MYALLOCNAME "libmymemory.so" -#define PROCNAME "/proc/*/maps" -#define REPLACEPOS 6 -#define PAGESIZE 4096 #include "snapshot-interface.h" #include #include @@ -13,6 +7,13 @@ #include #include +#define MYBINARYNAME "model" +#define MYLIBRARYNAME "libmodel.so" +#define MYALLOCNAME "libmymemory.so" +#define PROCNAME "/proc/*/maps" +#define REPLACEPOS 6 +#define PAGESIZE 4096 + snapshotStack * snapshotObject; /*This looks like it might leak memory... Subramanian should fix this. */ -- 2.34.1