diff options
| author | Edwin Mons <edwin.mons@isode.com> | 2019-11-19 13:36:05 (GMT) |
|---|---|---|
| committer | Edwin Mons <edwin.mons@isode.com> | 2019-11-19 13:58:45 (GMT) |
| commit | 261ba8d8595ed8cb90f9c4feb1d6ef642942bcba (patch) | |
| tree | c7e60d473509db8c4dbff5aa83fbde963d8dd75e /SwifTools | |
| parent | 697ae6ae84512a744958b24118197ec7bfdbc1f0 (diff) | |
| download | swift-261ba8d8595ed8cb90f9c4feb1d6ef642942bcba.zip swift-261ba8d8595ed8cb90f9c4feb1d6ef642942bcba.tar.bz2 | |
Remove std::endl from SWIFT_LOG calls
The std::endl is now added by ~Log, but only for output to stderr or a
log file. Calls to the Android logging system or manually set callbacks
will not include the newline in the logging output.
JIRA: SWIFT-430
Test-Information:
Unit tests pass on Debian 9
Checked that running Swift with logging to stderr still had a newline.
Change-Id: I096fdba78a3b8f87db2097951c28c528592183e8
Diffstat (limited to 'SwifTools')
| -rw-r--r-- | SwifTools/Application/ApplicationPathProvider.cpp | 4 | ||||
| -rw-r--r-- | SwifTools/Application/MacOSXApplicationPathProvider.cpp | 4 | ||||
| -rw-r--r-- | SwifTools/Application/UnixApplicationPathProvider.cpp | 4 | ||||
| -rw-r--r-- | SwifTools/AutoUpdater/SparkleAutoUpdater.mm | 4 | ||||
| -rw-r--r-- | SwifTools/AutoUpdater/SparkleAutoUpdaterDelegate.mm | 6 | ||||
| -rw-r--r-- | SwifTools/CrashReporter.cpp | 4 | ||||
| -rw-r--r-- | SwifTools/HunspellChecker.cpp | 4 | ||||
| -rw-r--r-- | SwifTools/Notifier/NotificationCenterNotifier.mm | 4 |
8 files changed, 17 insertions, 17 deletions
diff --git a/SwifTools/Application/ApplicationPathProvider.cpp b/SwifTools/Application/ApplicationPathProvider.cpp index 8b952bb..77457ef 100644 --- a/SwifTools/Application/ApplicationPathProvider.cpp +++ b/SwifTools/Application/ApplicationPathProvider.cpp | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2010-2016 Isode Limited. | 2 | * Copyright (c) 2010-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -25,7 +25,7 @@ boost::filesystem::path ApplicationPathProvider::getProfileDir(const std::string | |||
| 25 | boost::filesystem::create_directory(result); | 25 | boost::filesystem::create_directory(result); |
| 26 | } | 26 | } |
| 27 | catch (const boost::filesystem::filesystem_error& e) { | 27 | catch (const boost::filesystem::filesystem_error& e) { |
| 28 | SWIFT_LOG(error) << e.what() << std::endl; | 28 | SWIFT_LOG(error) << e.what(); |
| 29 | } | 29 | } |
| 30 | return result; | 30 | return result; |
| 31 | } | 31 | } |
diff --git a/SwifTools/Application/MacOSXApplicationPathProvider.cpp b/SwifTools/Application/MacOSXApplicationPathProvider.cpp index 684d8b2..d7b99b9 100644 --- a/SwifTools/Application/MacOSXApplicationPathProvider.cpp +++ b/SwifTools/Application/MacOSXApplicationPathProvider.cpp | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2010-2016 Isode Limited. | 2 | * Copyright (c) 2010-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -23,7 +23,7 @@ boost::filesystem::path MacOSXApplicationPathProvider::getDataDir() const { | |||
| 23 | boost::filesystem::create_directory(result); | 23 | boost::filesystem::create_directory(result); |
| 24 | } | 24 | } |
| 25 | catch (const boost::filesystem::filesystem_error& e) { | 25 | catch (const boost::filesystem::filesystem_error& e) { |
| 26 | SWIFT_LOG(error) << e.what() << std::endl; | 26 | SWIFT_LOG(error) << e.what(); |
| 27 | } | 27 | } |
| 28 | return result; | 28 | return result; |
| 29 | } | 29 | } |
diff --git a/SwifTools/Application/UnixApplicationPathProvider.cpp b/SwifTools/Application/UnixApplicationPathProvider.cpp index e455d23..a345766 100644 --- a/SwifTools/Application/UnixApplicationPathProvider.cpp +++ b/SwifTools/Application/UnixApplicationPathProvider.cpp | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2010-2016 Isode Limited. | 2 | * Copyright (c) 2010-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -54,7 +54,7 @@ boost::filesystem::path UnixApplicationPathProvider::getDataDir() const { | |||
| 54 | boost::filesystem::create_directories(dataPath); | 54 | boost::filesystem::create_directories(dataPath); |
| 55 | } | 55 | } |
| 56 | catch (const boost::filesystem::filesystem_error& e) { | 56 | catch (const boost::filesystem::filesystem_error& e) { |
| 57 | SWIFT_LOG(error) << "file system error: " << e.what() << std::endl; | 57 | SWIFT_LOG(error) << "file system error: " << e.what(); |
| 58 | } | 58 | } |
| 59 | return dataPath; | 59 | return dataPath; |
| 60 | } | 60 | } |
diff --git a/SwifTools/AutoUpdater/SparkleAutoUpdater.mm b/SwifTools/AutoUpdater/SparkleAutoUpdater.mm index b4a4c05..274ab3c 100644 --- a/SwifTools/AutoUpdater/SparkleAutoUpdater.mm +++ b/SwifTools/AutoUpdater/SparkleAutoUpdater.mm | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2016-2017 Isode Limited. | 2 | * Copyright (c) 2016-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -46,7 +46,7 @@ SparkleAutoUpdater::SparkleAutoUpdater(const std::string& appcastFeed) : d(new P | |||
| 46 | 46 | ||
| 47 | SWIFT_LOG(debug) << (canDoSilentUpdates ? | 47 | SWIFT_LOG(debug) << (canDoSilentUpdates ? |
| 48 | "The current running user has enough permissions to do a silent update." : | 48 | "The current running user has enough permissions to do a silent update." : |
| 49 | "The current running user has insufficient permissions to do a silent update.") << std::endl; | 49 | "The current running user has insufficient permissions to do a silent update."); |
| 50 | 50 | ||
| 51 | setAppcastFeed(appcastFeed); | 51 | setAppcastFeed(appcastFeed); |
| 52 | } | 52 | } |
diff --git a/SwifTools/AutoUpdater/SparkleAutoUpdaterDelegate.mm b/SwifTools/AutoUpdater/SparkleAutoUpdaterDelegate.mm index be58355..b9294d9 100644 --- a/SwifTools/AutoUpdater/SparkleAutoUpdaterDelegate.mm +++ b/SwifTools/AutoUpdater/SparkleAutoUpdaterDelegate.mm | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2016-2017 Isode Limited. | 2 | * Copyright (c) 2016-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -49,7 +49,7 @@ using namespace Swift; | |||
| 49 | - (void)updater:(SUUpdater *)updater failedToDownloadUpdate:(SUAppcastItem *)item error:(NSError *)error { | 49 | - (void)updater:(SUUpdater *)updater failedToDownloadUpdate:(SUAppcastItem *)item error:(NSError *)error { |
| 50 | (void)updater; | 50 | (void)updater; |
| 51 | (void)item; | 51 | (void)item; |
| 52 | SWIFT_LOG(error) << ns2StdString([error localizedDescription]) << std::endl; | 52 | SWIFT_LOG(error) << ns2StdString([error localizedDescription]); |
| 53 | onNewUpdateState(AutoUpdater::State::ErrorCheckingForUpdate); | 53 | onNewUpdateState(AutoUpdater::State::ErrorCheckingForUpdate); |
| 54 | } | 54 | } |
| 55 | 55 | ||
| @@ -71,7 +71,7 @@ using namespace Swift; | |||
| 71 | onNewUpdateState(AutoUpdater::State::NoUpdateAvailable); | 71 | onNewUpdateState(AutoUpdater::State::NoUpdateAvailable); |
| 72 | } | 72 | } |
| 73 | else { | 73 | else { |
| 74 | SWIFT_LOG(error) << ns2StdString([error localizedDescription]) << std::endl; | 74 | SWIFT_LOG(error) << ns2StdString([error localizedDescription]); |
| 75 | onNewUpdateState(AutoUpdater::State::ErrorCheckingForUpdate); | 75 | onNewUpdateState(AutoUpdater::State::ErrorCheckingForUpdate); |
| 76 | } | 76 | } |
| 77 | } | 77 | } |
diff --git a/SwifTools/CrashReporter.cpp b/SwifTools/CrashReporter.cpp index bf637c8..b02e73b 100644 --- a/SwifTools/CrashReporter.cpp +++ b/SwifTools/CrashReporter.cpp | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2012-2016 Isode Limited. | 2 | * Copyright (c) 2012-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -46,7 +46,7 @@ CrashReporter::CrashReporter(const boost::filesystem::path& path, const std::str | |||
| 46 | boost::filesystem::create_directories(path); | 46 | boost::filesystem::create_directories(path); |
| 47 | } | 47 | } |
| 48 | catch (const boost::filesystem::filesystem_error& e) { | 48 | catch (const boost::filesystem::filesystem_error& e) { |
| 49 | SWIFT_LOG(error) << "ERROR: " << e.what() << std::endl; | 49 | SWIFT_LOG(error) << "ERROR: " << e.what(); |
| 50 | } | 50 | } |
| 51 | } | 51 | } |
| 52 | 52 | ||
diff --git a/SwifTools/HunspellChecker.cpp b/SwifTools/HunspellChecker.cpp index 1de369b..019a4dc 100644 --- a/SwifTools/HunspellChecker.cpp +++ b/SwifTools/HunspellChecker.cpp | |||
| @@ -84,12 +84,12 @@ bool HunspellChecker::isAutomaticallyDetectingLanguage() { | |||
| 84 | void HunspellChecker::setActiveLanguage(const std::string& language) { | 84 | void HunspellChecker::setActiveLanguage(const std::string& language) { |
| 85 | auto dictionaries = detectedDictionaries(); | 85 | auto dictionaries = detectedDictionaries(); |
| 86 | if (dictionaries.find(language) != dictionaries.end()) { | 86 | if (dictionaries.find(language) != dictionaries.end()) { |
| 87 | SWIFT_LOG(debug) << "Initialized Hunspell with dic,aff files " << dictionaries[language].dicPath << " , " << dictionaries[language].affPath << std::endl; | 87 | SWIFT_LOG(debug) << "Initialized Hunspell with dic,aff files " << dictionaries[language].dicPath << " , " << dictionaries[language].affPath; |
| 88 | speller_ = std::unique_ptr<Hunspell>(new Hunspell(dictionaries[language].affPath.c_str(), dictionaries[language].dicPath.c_str())); | 88 | speller_ = std::unique_ptr<Hunspell>(new Hunspell(dictionaries[language].affPath.c_str(), dictionaries[language].dicPath.c_str())); |
| 89 | activeLangauge_ = language; | 89 | activeLangauge_ = language; |
| 90 | } | 90 | } |
| 91 | else { | 91 | else { |
| 92 | SWIFT_LOG(warning) << "Unsupported language '" << language << "'" << std::endl; | 92 | SWIFT_LOG(warning) << "Unsupported language '" << language << "'"; |
| 93 | } | 93 | } |
| 94 | } | 94 | } |
| 95 | 95 | ||
diff --git a/SwifTools/Notifier/NotificationCenterNotifier.mm b/SwifTools/Notifier/NotificationCenterNotifier.mm index 1538186..e6fdfb6 100644 --- a/SwifTools/Notifier/NotificationCenterNotifier.mm +++ b/SwifTools/Notifier/NotificationCenterNotifier.mm | |||
| @@ -1,5 +1,5 @@ | |||
| 1 | /* | 1 | /* |
| 2 | * Copyright (c) 2015-2016 Isode Limited. | 2 | * Copyright (c) 2015-2019 Isode Limited. |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * See the COPYING file for more information. | 4 | * See the COPYING file for more information. |
| 5 | */ | 5 | */ |
| @@ -88,7 +88,7 @@ void NotificationCenterNotifier::handleUserNotificationActivated(const std::stri | |||
| 88 | } | 88 | } |
| 89 | } | 89 | } |
| 90 | else { | 90 | else { |
| 91 | SWIFT_LOG(warning) << "Missing callback entry for activated notification. The activate notification may come from another instance." << std::endl; | 91 | SWIFT_LOG(warning) << "Missing callback entry for activated notification. The activate notification may come from another instance."; |
| 92 | } | 92 | } |
| 93 | } | 93 | } |
| 94 | 94 | ||
Swift