arch-x86: Stop "using namespace std"
authorGabe Black <gabe.black@gmail.com>
Thu, 21 Jan 2021 11:04:19 +0000 (03:04 -0800)
committerGabe Black <gabe.black@gmail.com>
Sat, 23 Jan 2021 22:47:13 +0000 (22:47 +0000)
Change-Id: I9763177bbd54abc97b99bd54a20a750e0adb5627
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/39557
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
Maintainer: Gabe Black <gabe.black@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/x86/bios/acpi.cc
src/arch/x86/bios/e820.cc
src/arch/x86/bios/intelmp.cc
src/arch/x86/bios/smbios.cc
src/arch/x86/isa/includes.isa
src/arch/x86/process.cc
src/arch/x86/remote_gdb.cc
src/arch/x86/types.cc

index 5ecb09bd42c82442941bf1408291f8ef8aa69374..20cf088807745072ff87462eb5712e2dd2d90d5f 100644 (file)
@@ -45,8 +45,6 @@
 #include "sim/byteswap.hh"
 #include "sim/sim_object.hh"
 
-using namespace std;
-
 const char X86ISA::ACPI::RSDP::signature[] = "RSD PTR ";
 
 X86ISA::ACPI::RSDP::RSDP(const Params &p) : SimObject(p), oemID(p.oem_id),
index a377703aaef2d3da722c1e3b362e80e466e39f23..a22f66c44f2038ba7cdd087c307258328d1a6354 100644 (file)
@@ -41,7 +41,6 @@
 #include "mem/port_proxy.hh"
 #include "sim/byteswap.hh"
 
-using namespace std;
 using namespace X86ISA;
 
 template<class T>
index 6378c48bb5936ac234a273a6ce98ceebb334da42..ba9ba202a7ff2cdd3d8ea2127e6d9ed2e2256e84 100644 (file)
@@ -63,8 +63,6 @@
 #include "params/X86IntelMPBusHierarchy.hh"
 #include "params/X86IntelMPCompatAddrSpaceMod.hh"
 
-using namespace std;
-
 const char X86ISA::IntelMP::FloatingPointer::signature[] = "_MP_";
 
 template<class T>
@@ -83,7 +81,7 @@ writeOutField(PortProxy& proxy, Addr addr, T val)
 }
 
 uint8_t
-writeOutString(PortProxy& proxy, Addr addr, string str, int length)
+writeOutString(PortProxy& proxy, Addr addr, std::string str, int length)
 {
     char cleanedString[length + 1];
     cleanedString[length] = 0;
@@ -210,7 +208,7 @@ X86ISA::IntelMP::ConfigTable::writeOut(PortProxy& proxy, Addr addr)
     proxy.writeBlob(addr + 43, &reserved, 1);
     checkSum += reserved;
 
-    vector<BaseConfigEntry *>::iterator baseEnt;
+    std::vector<BaseConfigEntry *>::iterator baseEnt;
     uint16_t offset = 44;
     for (baseEnt = baseEntries.begin();
             baseEnt != baseEntries.end(); baseEnt++) {
@@ -220,7 +218,7 @@ X86ISA::IntelMP::ConfigTable::writeOut(PortProxy& proxy, Addr addr)
     // We've found the end of the base table this point.
     checkSum += writeOutField(proxy, addr + 4, offset);
 
-    vector<ExtConfigEntry *>::iterator extEnt;
+    std::vector<ExtConfigEntry *>::iterator extEnt;
     uint16_t extOffset = 0;
     uint8_t extCheckSum = 0;
     for (extEnt = extEntries.begin();
index afe0f152fc6c5649ac40103f464c1cb0e1114715..ead87cf52d8400e9562e9859de339cc52d64c8b4 100644 (file)
@@ -48,8 +48,6 @@
 #include "params/X86SMBiosSMBiosTable.hh"
 #include "sim/byteswap.hh"
 
-using namespace std;
-
 const char X86ISA::SMBios::SMBiosTable::SMBiosHeader::anchorString[] = "_SM_";
 const uint8_t X86ISA::SMBios::SMBiosTable::
         SMBiosHeader::formattedArea[] = {0,0,0,0,0};
@@ -128,7 +126,7 @@ X86ISA::SMBios::SMBiosStructure::getStringLength()
 }
 
 int
-X86ISA::SMBios::SMBiosStructure::addString(const string &new_string)
+X86ISA::SMBios::SMBiosStructure::addString(const std::string &new_string)
 {
     stringFields = true;
     // If a string is empty, treat it as not existing. The index for empty
@@ -139,7 +137,7 @@ X86ISA::SMBios::SMBiosStructure::addString(const string &new_string)
     return strings.size();
 }
 
-string
+std::string
 X86ISA::SMBios::SMBiosStructure::readString(int n)
 {
     assert(n > 0 && n <= strings.size());
index d2098c3e9277124c8c9468b0358a813004f885b1..3bd7872b87fe1e8265b65777c2b8bbebd45f97fc 100644 (file)
@@ -126,6 +126,5 @@ output exec {{
 #include "sim/sim_exit.hh"
 
 using namespace X86ISA;
-using namespace std;
 }};
 
index 03e86418c9c7e85c339383602ad41e5d8dca435b..925c836197bd68ce59168fd154b841b7594c6f9d 100644 (file)
@@ -65,7 +65,6 @@
 #include "sim/syscall_return.hh"
 #include "sim/system.hh"
 
-using namespace std;
 using namespace X86ISA;
 
 template class MultiLevelPageTable<LongModePTE<47, 39>,
@@ -112,9 +111,9 @@ X86_64Process::X86_64Process(const ProcessParams &params,
     Addr next_thread_stack_base = stack_base - max_stack_size;
     Addr mmap_end = 0x7FFFF7FFF000ULL;
 
-    memState = make_shared<MemState>(this, brk_point, stack_base,
-                                     max_stack_size, next_thread_stack_base,
-                                     mmap_end);
+    memState = std::make_shared<MemState>(
+            this, brk_point, stack_base, max_stack_size,
+            next_thread_stack_base, mmap_end);
 }
 
 
@@ -139,9 +138,9 @@ I386Process::I386Process(const ProcessParams &params,
     Addr next_thread_stack_base = stack_base - max_stack_size;
     Addr mmap_end = 0xB7FFF000ULL;
 
-    memState = make_shared<MemState>(this, brk_point, stack_base,
-                                     max_stack_size, next_thread_stack_base,
-                                     mmap_end);
+    memState = std::make_shared<MemState>(
+            this, brk_point, stack_base, max_stack_size,
+            next_thread_stack_base, mmap_end);
 }
 
 void
@@ -714,7 +713,7 @@ X86Process::argsInit(int pageSize,
 
     std::vector<AuxVector<IntType>> auxv = extraAuxvs;
 
-    string filename;
+    std::string filename;
     if (argv.size() < 1)
         filename = "";
     else
@@ -850,7 +849,7 @@ X86Process::argsInit(int pageSize,
     const int numRandomBytes = 16;
     int aux_data_size = numRandomBytes;
 
-    string platform = "x86_64";
+    std::string platform = "x86_64";
     aux_data_size += platform.size() + 1;
 
     int env_data_size = 0;
index 2f38fd5f32569a80f7bf4aa2a25316a3e85de3c5..c2d622e2d961349c6f50a2b191ca241dc7076c46 100644 (file)
@@ -58,7 +58,6 @@
 #include "mem/page_table.hh"
 #include "sim/full_system.hh"
 
-using namespace std;
 using namespace X86ISA;
 
 RemoteGDB::RemoteGDB(System *_system, ThreadContext *c, int _port) :
index 85884674818b13af26a9fff19349095d8a53aaea..58066233affa665038afef04567f45029dbc3206 100644 (file)
 #include "sim/serialize.hh"
 
 using namespace X86ISA;
-using namespace std;
 
 template <>
 void
-paramOut(CheckpointOut &cp, const string &name, ExtMachInst const &machInst)
+paramOut(CheckpointOut &cp, const std::string &name,
+        ExtMachInst const &machInst)
 {
     // Prefixes
     paramOut(cp, name + ".legacy", (uint8_t)machInst.legacy);
@@ -66,7 +66,7 @@ paramOut(CheckpointOut &cp, const string &name, ExtMachInst const &machInst)
 
 template <>
 void
-paramIn(CheckpointIn &cp, const string &name, ExtMachInst &machInst)
+paramIn(CheckpointIn &cp, const std::string &name, ExtMachInst &machInst)
 {
     uint8_t temp8;
     // Prefixes