Per code review:

* get rid of (void) construct in function declarations
* make toString a const member
* add a default implementation of toString for Win32


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@17873 91177308-0d34-0410-b5e6-96231b3b80d8

4 files changed, 10 insertions, 2 deletions

diff --git a/lib/System/Unix/TimeValue.cpp b/lib/System/Unix/TimeValue.cpp
index d140af9add..0642aa58c3 100644
--- a/lib/System/Unix/TimeValue.cpp
+++ b/lib/System/Unix/TimeValue.cpp
@@ -25,7 +25,7 @@ namespace llvm {
   using namespace sys;
 
 
-std::string TimeValue::toString() {
+std::string TimeValue::toString() const {
   char buffer[32];
 
   time_t ourTime = time_t(this->toEpochTime());
diff --git a/lib/System/Unix/TimeValue.inc b/lib/System/Unix/TimeValue.inc
index d140af9add..0642aa58c3 100644
--- a/lib/System/Unix/TimeValue.inc
+++ b/lib/System/Unix/TimeValue.inc
@@ -25,7 +25,7 @@ namespace llvm {
   using namespace sys;
 
 
-std::string TimeValue::toString() {
+std::string TimeValue::toString() const {
   char buffer[32];
 
   time_t ourTime = time_t(this->toEpochTime());
diff --git a/lib/System/Win32/TimeValue.cpp b/lib/System/Win32/TimeValue.cpp
index 433b948748..eab776fc17 100644
--- a/lib/System/Win32/TimeValue.cpp
+++ b/lib/System/Win32/TimeValue.cpp
@@ -30,6 +30,10 @@ TimeValue TimeValue::now() {
     static_cast<TimeValue::NanoSecondsType>( (ft % 10000000) * 100) );
 }
 
+std::string TimeValue::toString() const {
+  return "Don't know how to conver time on Win32";
+}
+
 // vim: sw=2 smartindent smarttab tw=80 autoindent expandtab
 
 }
diff --git a/lib/System/Win32/TimeValue.inc b/lib/System/Win32/TimeValue.inc
index 433b948748..eab776fc17 100644
--- a/lib/System/Win32/TimeValue.inc
+++ b/lib/System/Win32/TimeValue.inc
@@ -30,6 +30,10 @@ TimeValue TimeValue::now() {
     static_cast<TimeValue::NanoSecondsType>( (ft % 10000000) * 100) );
 }
 
+std::string TimeValue::toString() const {
+  return "Don't know how to conver time on Win32";
+}
+
 // vim: sw=2 smartindent smarttab tw=80 autoindent expandtab
 
 }