From 4336abba16bcd2db4ea0606119483ccb6d86df28 Mon Sep 17 00:00:00 2001
From: Frederic Suter <frederic.suter@cc.in2p3.fr>
Date: Thu, 8 Jun 2017 08:45:03 +0200
Subject: [PATCH] use ordered map when order is needed

---
 include/simgrid/s4u/Storage.hpp                               | 2 +-
 src/s4u/s4u_storage.cpp                                       | 4 ++--
 teshsuite/s4u/storage_client_server/storage_client_server.cpp | 2 +-
 .../s4u/storage_client_server/storage_client_server.tesh      | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/simgrid/s4u/Storage.hpp b/include/simgrid/s4u/Storage.hpp
index 85c16ae3ce..12bfe853a3 100644
--- a/include/simgrid/s4u/Storage.hpp
+++ b/include/simgrid/s4u/Storage.hpp
@@ -17,7 +17,7 @@
 namespace simgrid {
 namespace s4u {
 
-std::unordered_map<std::string, Storage*>* allStorages();
+std::map<std::string, Storage*>* allStorages();
 
 XBT_PUBLIC_CLASS Storage
 {
diff --git a/src/s4u/s4u_storage.cpp b/src/s4u/s4u_storage.cpp
index e3e776c928..5e9d200358 100644
--- a/src/s4u/s4u_storage.cpp
+++ b/src/s4u/s4u_storage.cpp
@@ -11,10 +11,10 @@
 
 namespace simgrid {
 namespace s4u {
-std::unordered_map<std::string, Storage*>* allStorages()
+std::map<std::string, Storage*>* allStorages()
 {
   std::unordered_map<std::string, surf::StorageImpl*>* map = surf::StorageImpl::storagesMap();
-  std::unordered_map<std::string, Storage*>* res           = new std::unordered_map<std::string, Storage*>;
+  std::map<std::string, Storage*>* res                     = new std::map<std::string, Storage*>;
   for (auto s : *map)
     res->insert({s.first, &(s.second->piface_)}); // Convert each entry into its interface
 
diff --git a/teshsuite/s4u/storage_client_server/storage_client_server.cpp b/teshsuite/s4u/storage_client_server/storage_client_server.cpp
index b801f7c8d2..b2fdc637b2 100644
--- a/teshsuite/s4u/storage_client_server/storage_client_server.cpp
+++ b/teshsuite/s4u/storage_client_server/storage_client_server.cpp
@@ -95,7 +95,7 @@ static void get_set_storage_data(const char* storage_name)
 
 static void dump_platform_storages()
 {
-  std::unordered_map<std::string, simgrid::s4u::Storage*>* storages = simgrid::s4u::allStorages();
+  std::map<std::string, simgrid::s4u::Storage*>* storages = simgrid::s4u::allStorages();
 
   for (auto storage : *storages) {
     XBT_INFO("Storage %s is attached to %s", storage.first.c_str(), storage.second->host());
diff --git a/teshsuite/s4u/storage_client_server/storage_client_server.tesh b/teshsuite/s4u/storage_client_server/storage_client_server.tesh
index c183aa2a09..d334f6b9f4 100644
--- a/teshsuite/s4u/storage_client_server/storage_client_server.tesh
+++ b/teshsuite/s4u/storage_client_server/storage_client_server.tesh
@@ -102,7 +102,7 @@ $ ./storage_client_server$EXEEXT ${srcdir:=.}/../../../examples/platforms/storag
 > [  1.207952] (server@alice) 	\Windows\winhlp32.exe size: 10752 bytes
 > [  1.207952] (server@alice) 	\Windows\write.exe size: 10752 bytes
 > [  1.207952] (server@alice) Storage Disk1 is attached to bob
-> [  1.207952] (server@alice) Storage Disk3 is attached to carl
 > [  1.207952] (server@alice) Storage Disk2 is attached to alice
+> [  1.207952] (server@alice) Storage Disk3 is attached to carl
 > [  1.207952] (server@alice) Storage Disk4 is attached to denise
 > [  1.207952] (maestro@) Simulated time: 1.20795
-- 
2.20.1