From b42e7645d3eaab0a2ab5e618cd322215f8164390 Mon Sep 17 00:00:00 2001
From: Brian Norris <banorris@uci.edu>
Date: Fri, 25 May 2012 18:10:58 -0700
Subject: [PATCH] snapshot-interface: cleanup interface header

The MyString type does not belong in an interface header. It's only used in one
file, so move its typedef locally.

Remove/move a bunch of misplaced #includes.

There's an unnecessary forward declaration of class snapshotStack.
---
 snapshot-interface.cc | 5 +++++
 snapshot-interface.h  | 9 ---------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/snapshot-interface.cc b/snapshot-interface.cc
index 76cb8f4..b0df836 100644
--- a/snapshot-interface.cc
+++ b/snapshot-interface.cc
@@ -5,7 +5,10 @@
 #include <sys/types.h>
 #include <sstream>
 #include <cstring>
+#include <string>
 #include <cassert>
+#include <vector>
+#include <utility>
 
 #define MYBINARYNAME "model"
 #define MYLIBRARYNAME "libmodel.so"
@@ -13,6 +16,8 @@
 #define REPLACEPOS		6
 #define PAGESIZE 4096
 
+typedef std::basic_string<char, std::char_traits<char>, MyAlloc<char> > MyString;
+
 snapshotStack * snapshotObject;
 
 /*This looks like it might leak memory...  Subramanian should fix this. */
diff --git a/snapshot-interface.h b/snapshot-interface.h
index cbebcd5..0fab53b 100644
--- a/snapshot-interface.h
+++ b/snapshot-interface.h
@@ -2,16 +2,7 @@
 #define __SNAPINTERFACE_H
 #include "snapshot.h"
 #include "mymemory.h"
-#include <vector>
-#include <utility>
-#include <string>
-#include <map>
-#include <set>
 #include "snapshot.h"
-#include "libthreads.h"
-
-class snapshotStack;
-typedef std::basic_string<char, std::char_traits<char>, MyAlloc<char> > MyString;
 
 void SnapshotGlobalSegments();
 
-- 
2.34.1