diff options
author | David Kalnischkies <david@kalnischkies.de> | 2016-03-12 15:49:54 +0100 |
---|---|---|
committer | David Kalnischkies <david@kalnischkies.de> | 2016-04-25 15:35:52 +0200 |
commit | 82153438235144c485a8539b5c035e67bbef96e4 (patch) | |
tree | fb82b0c81c52d2d70dc734bb63bcb2ab22547b8e /apt-pkg/contrib | |
parent | 3383ef4d30b3fb1057e21f5598d3128b9afbe34d (diff) | |
download | apt-82153438235144c485a8539b5c035e67bbef96e4.tar.gz |
properly format multiline error messages
Diffstat (limited to 'apt-pkg/contrib')
-rw-r--r-- | apt-pkg/contrib/error.h | 27 |
1 files changed, 21 insertions, 6 deletions
diff --git a/apt-pkg/contrib/error.h b/apt-pkg/contrib/error.h index 80d731ac9..e56999b14 100644 --- a/apt-pkg/contrib/error.h +++ b/apt-pkg/contrib/error.h @@ -316,15 +316,30 @@ private: /*{{{*/ Item(char const *Text, MsgType const &Type) : Text(Text), Type(Type) {}; - friend std::ostream& operator<< (std::ostream &out, Item i) { + APT_HIDDEN friend std::ostream& operator<< (std::ostream &out, Item i) { switch(i.Type) { case FATAL: - case ERROR: out << "E"; break; - case WARNING: out << "W"; break; - case NOTICE: out << "N"; break; - case DEBUG: out << "D"; break; + case ERROR: out << 'E'; break; + case WARNING: out << 'W'; break; + case NOTICE: out << 'N'; break; + case DEBUG: out << 'D'; break; } - return out << ": " << i.Text; + out << ": "; + std::string::size_type line_start = 0; + std::string::size_type line_end; + while ((line_end = i.Text.find_first_of("\n\r", line_start)) != std::string::npos) { + if (line_start != 0) + out << std::endl << " "; + out << i.Text.substr(line_start, line_end - line_start); + line_start = i.Text.find_first_not_of("\n\r", line_end + 1); + if (line_start == std::string::npos) + break; + } + if (line_start == 0) + out << i.Text; + else if (line_start != std::string::npos) + out << std::endl << " " << i.Text.substr(line_start); + return out; } }; |