Merged PR 2064: Use ostringstream to replace stringstream

Use ostringstream to replace stringstream

Author: snnn

lotus/core/framework/bfc_arena.h

@@ -53,7 +53,7 @@ struct AllocatorStats {
   }
 
   std::string DebugString() const {
-    std::stringstream ss;
+    std::ostringstream ss;
     ss << "Limit:          " << this->bytes_limit << "\n"
        << "InUse:         " << this->bytes_in_use << "\n"
        << "TotalAllocate: " << this->total_allocated_bytes_ << "\n"
@@ -162,7 +162,7 @@ class BFCArena : public IArenaAllocator {
 
     std::string DebugString(BFCArena* a,
                             bool recurse) {
-      std::stringstream ss;
+      std::ostringstream ss;
       ss << "  Size: " << size << " | Requested Size: " << requested_size << " | in_use: " << in_use();
       if (recurse && prev != BFCArena::kInvalidChunkHandle) {
         Chunk* p = a->ChunkFromHandle(prev);

lotus/core/framework/inference_session.cc

@@ -780,7 +780,7 @@ class InferenceSession::Impl {
   }
 
   void StartProfiling(const std::string& file_prefix) {
-    std::stringstream ss;
+    std::ostringstream ss;
     ss << file_prefix << "_" << GetCurrentTimeString() << ".json";
     session_profiler_.StartProfiling(session_logger_, ss.str());
   }

lotus/core/graph/schema_registry.cc

@@ -59,7 +59,7 @@ Lotus::Common::Status LotusOpSchemaRegistry::RegisterOpSchemaInternal(ONNX_NAMES
 
   if (map_[op_name][op_domain].count(ver)) {
     const auto& schema = map_[op_name][op_domain][ver];
-    std::stringstream ostream;
+    std::ostringstream ostream;
     ostream << "Trying to register schema with name " << op_name
             << " (domain: " << op_domain << " version: " << ver
             << ") from file " << op_schema.file() << " line "
@@ -71,7 +71,7 @@ Lotus::Common::Status LotusOpSchemaRegistry::RegisterOpSchemaInternal(ONNX_NAMES
 
   auto ver_range_it = domain_version_range_map_.find(op_domain);
   if (ver_range_it == domain_version_range_map_.end()) {
-    std::stringstream ostream;
+    std::ostringstream ostream;
     ostream << "Trying to register schema with name " << op_name
             << " (domain: " << op_domain << " version: " << ver
             << ") from file " << op_schema.file() << " line "
@@ -80,7 +80,7 @@ Lotus::Common::Status LotusOpSchemaRegistry::RegisterOpSchemaInternal(ONNX_NAMES
     return Lotus::Common::Status(Lotus::Common::LOTUS, Lotus::Common::INVALID_ARGUMENT, ostream.str());
   }
   if (ver > ver_range_it->second.opset_version) {
-    std::stringstream ostream;
+    std::ostringstream ostream;
     ostream
         << "Trying to register schema with name " << op_name
         << " (domain: " << op_domain << " version: " << ver

lotus/test/framework/TestAllocatorManager.cc

@@ -4,7 +4,7 @@ namespace Lotus {
 namespace Test {
 
 static std::string GetAllocatorId(const std::string& name, const int id, const bool isArena) {
-  std::stringstream ss;
+  std::ostringstream ss;
   if (isArena)
     ss << "arena_";
   else

lotus/test/framework/tensor_test.cc

@@ -200,7 +200,7 @@ TEST(TensorTest, ConvertToString) {
 
   EXPECT_EQ(shape.ToString(), "{2,3,4}");
 
-  std::stringstream ss;
+  std::ostringstream ss;
   ss << shape;
   EXPECT_EQ(ss.str(), "{2,3,4}");
 }