From: Martin Quinson Date: Thu, 11 Jul 2019 06:54:28 +0000 (+0200) Subject: mc/compare: another round of tiny cleanups X-Git-Tag: v3.24~320 X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/c21e80010fc3e012cdceac59b8ab626e5ba9756d mc/compare: another round of tiny cleanups --- diff --git a/src/mc/compare.cpp b/src/mc/compare.cpp index 151fda1521..6948be571b 100644 --- a/src/mc/compare.cpp +++ b/src/mc/compare.cpp @@ -12,29 +12,14 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_compare, xbt, "Logging specific to mc_compare in mc"); +using simgrid::mc::remote; + namespace simgrid { namespace mc { -struct HeapLocation; -typedef std::array HeapLocationPair; -typedef std::set HeapLocationPairs; -struct HeapArea; -struct ProcessComparisonState; -struct StateComparator; - -static int compare_heap_area(StateComparator& state, const void* area1, const void* area2, Snapshot* snapshot1, - Snapshot* snapshot2, HeapLocationPairs* previous, Type* type, int pointer_level); -} -} - -using simgrid::mc::remote; - /*********************************** Heap comparison ***********************************/ /***************************************************************************************/ -namespace simgrid { -namespace mc { - class HeapLocation { public: int block_ = 0; @@ -53,13 +38,16 @@ public: } }; +typedef std::array HeapLocationPair; +typedef std::set HeapLocationPairs; + +struct ProcessComparisonState; +struct StateComparator; + static inline HeapLocationPair makeHeapLocationPair(int block1, int fragment1, int block2, int fragment2) { - return simgrid::mc::HeapLocationPair{{ - simgrid::mc::HeapLocation(block1, fragment1), - simgrid::mc::HeapLocation(block2, fragment2) - }}; + return HeapLocationPair{{HeapLocation(block1, fragment1), HeapLocation(block2, fragment2)}}; } class HeapArea : public HeapLocation { @@ -84,6 +72,9 @@ public: void initHeapInformation(xbt_mheap_t heap, std::vector* i); }; +static int compare_heap_area(StateComparator& state, const void* area1, const void* area2, Snapshot* snapshot1, + Snapshot* snapshot2, HeapLocationPairs* previous, Type* type, int pointer_level); + namespace { /** A hash which works with more stuff