Use std::string for TBD/missing functionality reporting.

We can have multiple instances of the same string, so comparing
const char* is not guaranteed working.

Fixed the failure on VS 2013 with Debug build.

1 files changed, 4 insertions, 4 deletions

diff --git a/SPIRV/Logger.h b/SPIRV/Logger.h
index 2b9eb0d5..15b5e354 100644
--- a/SPIRV/Logger.h
+++ b/SPIRV/Logger.h
@@ -48,9 +48,9 @@ public:
     SpvBuildLogger(const SpvBuildLogger&) = delete;
 
     // Registers a TBD functionality.
-    void tbdFunctionality(const char* f);
+    void tbdFunctionality(const std::string& f);
     // Registers a missing functionality.
-    void missingFunctionality(const char* f);
+    void missingFunctionality(const std::string& f);
 
     // Logs a warning.
     void warning(const std::string& w) { warnings.push_back(w); }
@@ -62,8 +62,8 @@ public:
     std::string getAllMessages() const;
 
 private:
-    std::vector<const char*> tbdFeatures;
-    std::vector<const char*> missingFeatures;
+    std::vector<std::string> tbdFeatures;
+    std::vector<std::string> missingFeatures;
     std::vector<std::string> warnings;
     std::vector<std::string> errors;
 };