Browse Source

2007-06-12 Tatsuhiro Tsujikawa <tujikawa at rednoah dot com>

	Changed format of log file.
	* src/SimpleLogger.cc
Tatsuhiro Tsujikawa 18 years ago
parent
commit
ba6e5ac7e5
5 changed files with 15 additions and 9 deletions
  1. 5 0
      ChangeLog
  2. 1 1
      src/BtPieceMessage.cc
  3. 1 2
      src/RequestGroupMan.cc
  4. 7 5
      src/SimpleLogger.cc
  5. 1 1
      src/SimpleLogger.h

+ 5 - 0
ChangeLog

@@ -1,3 +1,8 @@
+2007-06-12  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
+
+	Changed format of log file.
+	* src/SimpleLogger.cc
+
 2007-06-10  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
 
 	* src/AbstractCommand.cc

+ 1 - 1
src/BtPieceMessage.cc

@@ -201,7 +201,7 @@ void BtPieceMessage::onNewPiece(const PieceHandle& piece) {
 }
 
 void BtPieceMessage::onWrongPiece(const PieceHandle& piece) {
-  logger->error(MSG_GOT_WRONG_PIECE, cuid, piece->getIndex());
+  logger->info(MSG_GOT_WRONG_PIECE, cuid, piece->getIndex());
   erasePieceOnDisk(piece);
   piece->clearAllBlock();
   requestFactory->removeTargetPiece(piece);

+ 1 - 2
src/RequestGroupMan.cc

@@ -105,10 +105,9 @@ void RequestGroupMan::showDownloadResults(ostream& o) const
     <<_("Download Results:") << "\n"
     << "idx|stat|path/URI" << "\n"
     << "===+====+======================================================================" << "\n";
-  int32_t count = 0;
   for(RequestGroups::const_iterator itr = _requestGroups.begin();
       itr != _requestGroups.end(); ++itr) {
-    o << setw(3) << ++count << "|";
+    o << setw(3) << (*itr)->getGID() << "|";
     if((*itr)->downloadFinished()) {
       o << "OK  ";
     } else {

+ 7 - 5
src/SimpleLogger.cc

@@ -81,7 +81,7 @@ void SimpleLogger::setStdout(int level, bool enabled) {
 }
 
 void SimpleLogger::writeHeader(FILE* file, string date, string level) const {
-  fprintf(file, "%s - %s - ", date.c_str(), level.c_str());
+  fprintf(file, "%s %s - ", date.c_str(), level.c_str());
 }
 
 void SimpleLogger::writeLog(FILE* file, int level, const char* msg, va_list ap, Exception* e, bool printHeader) const
@@ -105,9 +105,11 @@ void SimpleLogger::writeLog(FILE* file, int level, const char* msg, va_list ap,
     levelStr = "INFO";
   }
   time_t now = time(NULL);
-  char datestr[26];
-  ctime_r(&now, datestr);
-  datestr[strlen(datestr)-1] = '\0';
+  char datestr[20];
+  struct tm tm;
+  localtime_r(&now, &tm);
+  strftime(datestr, sizeof(datestr), "%Y-%m-%d %H:%M:%S", &tm);
+
   // TODO a quick hack not to print header in console
   if(printHeader) {
     writeHeader(file, datestr, levelStr);
@@ -127,7 +129,7 @@ void SimpleLogger::writeFile(int level, const char* msg, va_list ap, Exception*
   writeLog(file, level, msg, ap, e);
   if(stdoutField&level) {
     fprintf(stdout, "\n");
-    writeLog(stdout, level, msg, ap, e, false);
+    writeLog(stdout, level, msg, ap, e);
   }
 }
 

+ 1 - 1
src/SimpleLogger.h

@@ -41,7 +41,7 @@ class SimpleLogger:public Logger {
 private:
   void writeFile(int level, const char* msg, va_list ap, Exception* e = 0) const;
   void writeHeader(FILE* file, string date, string level) const;
-  void writeLog(FILE* file, int level, const char* msg, va_list ap, Exception* e = 0, bool printHeader = false) const;
+  void writeLog(FILE* file, int level, const char* msg, va_list ap, Exception* e = 0, bool printHeader = true) const;
   FILE* file;
   int stdoutField;
 public: