integrated changes from 1.3.6

Util/src/AbstractConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // AbstractConfiguration.cpp
 //
-// $Id: //poco/svn/Util/src/AbstractConfiguration.cpp#1 $
+// $Id: //poco/Main/Util/src/AbstractConfiguration.cpp#7 $
 //
 // Library: Util
 // Package: Configuration

Util/src/ConfigurationMapper.cpp

@@ -1,7 +1,7 @@
 //
 // ConfigurationMapper.cpp
 //
-// $Id: //poco/svn/Util/src/ConfigurationMapper.cpp#1 $
+// $Id: //poco/Main/Util/src/ConfigurationMapper.cpp#6 $
 //
 // Library: Util
 // Package: Configuration

Util/src/ConfigurationView.cpp

@@ -1,7 +1,7 @@
 //
 // ConfigurationView.cpp
 //
-// $Id: //poco/svn/Util/src/ConfigurationView.cpp#1 $
+// $Id: //poco/Main/Util/src/ConfigurationView.cpp#7 $
 //
 // Library: Util
 // Package: Configuration

Util/src/FilesystemConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // FilesystemConfiguration.cpp
 //
-// $Id: //poco/svn/Util/src/FilesystemConfiguration.cpp#1 $
+// $Id: //poco/Main/Util/src/FilesystemConfiguration.cpp#8 $
 //
 // Library: Util
 // Package: Configuration

Util/src/HelpFormatter.cpp

@@ -1,7 +1,7 @@
 //
 // HelpFormatter.cpp
 //
-// $Id: //poco/svn/Util/src/HelpFormatter.cpp#1 $
+// $Id: //poco/Main/Util/src/HelpFormatter.cpp#8 $
 //
 // Library: Util
 // Package: Options

Util/src/IniFileConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // IniFileConfiguration.cpp
 //
-// $Id: //poco/svn/Util/src/IniFileConfiguration.cpp#1 $
+// $Id: //poco/Main/Util/src/IniFileConfiguration.cpp#9 $
 //
 // Library: Util
 // Package: Configuration

Util/src/IntValidator.cpp

@@ -1,7 +1,7 @@
 //
 // IntValidator.cpp
 //
-// $Id: //poco/svn/Util/src/IntValidator.cpp#1 $
+// $Id: //poco/Main/Util/src/IntValidator.cpp#1 $
 //
 // Library: Util
 // Package: Options

Util/src/LayeredConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // LayeredConfiguration.cpp
 //
-// $Id: //poco/svn/Util/src/LayeredConfiguration.cpp#1 $
+// $Id: //poco/Main/Util/src/LayeredConfiguration.cpp#8 $
 //
 // Library: Util
 // Package: Configuration

Util/src/LoggingConfigurator.cpp

@@ -1,7 +1,7 @@
 //
 // LoggingConfigurator.cpp
 //
-// $Id: //poco/svn/Util/src/LoggingConfigurator.cpp#1 $
+// $Id: //poco/Main/Util/src/LoggingConfigurator.cpp#12 $
 //
 // Library: Util
 // Package: Configuration

Util/src/LoggingSubsystem.cpp

@@ -1,7 +1,7 @@
 //
 // LoggingSubsystem.cpp
 //
-// $Id: //poco/svn/Util/src/LoggingSubsystem.cpp#1 $
+// $Id: //poco/Main/Util/src/LoggingSubsystem.cpp#8 $
 //
 // Library: Util
 // Package: Application

Util/src/MapConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // MapConfiguration.cpp
 //
-// $Id: //poco/svn/Util/src/MapConfiguration.cpp#1 $
+// $Id: //poco/Main/Util/src/MapConfiguration.cpp#6 $
 //
 // Library: Util
 // Package: Configuration

Util/src/Option.cpp

@@ -1,7 +1,7 @@
 //
 // Option.cpp
 //
-// $Id: //poco/svn/Util/src/Option.cpp#2 $
+// $Id: //poco/Main/Util/src/Option.cpp#12 $
 //
 // Library: Util
 // Package: Options

Util/src/OptionCallback.cpp

@@ -1,7 +1,7 @@
 //
 // OptionCallback.cpp
 //
-// $Id: //poco/svn/Util/src/OptionCallback.cpp#1 $
+// $Id: //poco/Main/Util/src/OptionCallback.cpp#2 $
 //
 // Library: Util
 // Package: Options

Util/src/OptionException.cpp

@@ -1,7 +1,7 @@
 //
 // OptionException.cpp
 //
-// $Id: //poco/svn/Util/src/OptionException.cpp#1 $
+// $Id: //poco/Main/Util/src/OptionException.cpp#8 $
 //
 // Library: Util
 // Package: Options

Util/src/OptionProcessor.cpp

@@ -1,7 +1,7 @@
 //
 // OptionProcessor.cpp
 //
-// $Id: //poco/svn/Util/src/OptionProcessor.cpp#1 $
+// $Id: //poco/Main/Util/src/OptionProcessor.cpp#8 $
 //
 // Library: Util
 // Package: Options

Util/src/OptionSet.cpp

@@ -1,7 +1,7 @@
 //
 // OptionSet.cpp
 //
-// $Id: //poco/svn/Util/src/OptionSet.cpp#1 $
+// $Id: //poco/Main/Util/src/OptionSet.cpp#9 $
 //
 // Library: Util
 // Package: Options
@@ -89,7 +89,7 @@ bool OptionSet::hasOption(const std::string& name, bool matchShort) const
 	bool found = false;
 	for (Iterator it = _options.begin(); it != _options.end(); ++it)
 	{
-		if (matchShort && it->matchesShort(name) || !matchShort && it->matchesFull(name))
+		if ((matchShort && it->matchesShort(name)) || (!matchShort && it->matchesFull(name)))
 		{
 			if (!found)
 				found = true;
@@ -106,7 +106,7 @@ const Option& OptionSet::getOption(const std::string& name, bool matchShort) con
 	const Option* pOption = 0;
 	for (Iterator it = _options.begin(); it != _options.end(); ++it)
 	{
-		if (matchShort && it->matchesShort(name) || !matchShort && it->matchesPartial(name))
+		if ((matchShort && it->matchesShort(name)) || (!matchShort && it->matchesPartial(name)))
 		{
 			if (!pOption)
 			{

Util/src/RegExpValidator.cpp

@@ -1,7 +1,7 @@
 //
 // RegExpValidator.cpp
 //
-// $Id: //poco/svn/Util/src/RegExpValidator.cpp#1 $
+// $Id: //poco/Main/Util/src/RegExpValidator.cpp#2 $
 //
 // Library: Util
 // Package: Options

Util/src/ServerApplication.cpp

@@ -1,7 +1,7 @@
 //
 // ServerApplication.cpp
 //
-// $Id: //poco/Main/Util/src/ServerApplication.cpp#28 $
+// $Id: //poco/Main/Util/src/ServerApplication.cpp#29 $
 //
 // Library: Util
 // Package: Application
@@ -423,7 +423,8 @@ int ServerApplication::run(int argc, char** argv)
 		init(argc, argv);
 		if (runAsDaemon)
 		{
-			chdir("/");
+			int rc = chdir("/");
+			if (rc != 0) return EXIT_OSERR;
 		}
 	}
 	catch (Exception& exc)

Util/src/Subsystem.cpp

@@ -1,7 +1,7 @@
 //
 // Subsystem.cpp
 //
-// $Id: //poco/svn/Util/src/Subsystem.cpp#1 $
+// $Id: //poco/Main/Util/src/Subsystem.cpp#8 $
 //
 // Library: Util
 // Package: Application

Util/src/Timer.cpp

@@ -1,7 +1,7 @@
 //
 // Timer.cpp
 //
-// $Id: //poco/Main/Util/src/Timer.cpp#1 $
+// $Id: //poco/Main/Util/src/Timer.cpp#2 $
 //
 // Library: Util
 // Package: Timer
@@ -178,7 +178,7 @@ public:
 		if (!task()->isCancelled())
 		{
 			Poco::Timestamp nextExecution;
-			nextExecution += _interval*1000;
+			nextExecution += static_cast<Poco::Timestamp::TimeDiff>(_interval)*1000;
 			queue().enqueueNotification(this, nextExecution);
 			duplicate();
 		}
@@ -210,7 +210,7 @@ public:
 		if (!task()->isCancelled())
 		{
 			Poco::Timestamp nextExecution(task()->lastExecution());
-			nextExecution += _interval*1000;
+			nextExecution += static_cast<Poco::Timestamp::TimeDiff>(_interval)*1000;
 			queue().enqueueNotification(this, nextExecution);
 			duplicate();
 		}
@@ -259,7 +259,7 @@ void Timer::schedule(TimerTask::Ptr pTask, Poco::Timestamp time)
 void Timer::schedule(TimerTask::Ptr pTask, long delay, long interval)
 {
 	Poco::Timestamp time;
-	time += delay*1000;
+	time += static_cast<Poco::Timestamp::TimeDiff>(delay)*1000;
 	schedule(pTask, time, interval);
 }
 
@@ -273,7 +273,7 @@ void Timer::schedule(TimerTask::Ptr pTask, Poco::Timestamp time, long interval)
 void Timer::scheduleAtFixedRate(TimerTask::Ptr pTask, long delay, long interval)
 {
 	Poco::Timestamp time;
-	time += delay*1000;
+	time += static_cast<Poco::Timestamp::TimeDiff>(delay)*1000;
 	scheduleAtFixedRate(pTask, time, interval);
 }
 

Util/src/Validator.cpp

@@ -1,7 +1,7 @@
 //
 // Validator.cpp
 //
-// $Id: //poco/svn/Util/src/Validator.cpp#1 $
+// $Id: //poco/Main/Util/src/Validator.cpp#1 $
 //
 // Library: Util
 // Package: Options

Util/src/WinRegistryConfiguration.cpp

@@ -1,7 +1,7 @@
 //
 // WinRegistryConfiguration.cpp
 //
-// $Id: //poco/1.3/Util/src/WinRegistryConfiguration.cpp#3 $
+// $Id: //poco/Main/Util/src/WinRegistryConfiguration.cpp#8 $
 //
 // Library: Util
 // Package: Windows

Util/src/WinRegistryKey.cpp

@@ -1,7 +1,7 @@
 //
 // WinRegistryKey.cpp
 //
-// $Id: //poco/1.3/Util/src/WinRegistryKey.cpp#5 $
+// $Id: //poco/Main/Util/src/WinRegistryKey.cpp#14 $
 //
 // Library: Util
 // Package: Windows

Util/src/WinService.cpp

@@ -1,7 +1,7 @@
 //
 // WinService.cpp
 //
-// $Id: //poco/svn/Util/src/WinService.cpp#1 $
+// $Id: //poco/Main/Util/src/WinService.cpp#10 $
 //
 // Library: Util
 // Package: Windows