[PATCH spice-streaming-agent v3 4/4] better error message when opening streaming device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Signed-off-by: Lukáš Hrázký <lhrazky@xxxxxxxxxx>
---
(rant alert :) I find this alignment quite unfortunate, while in this
case it's all fine and pretty, if there was more stuff in there, with
such big indent there is little space for it which makes it
unnecessarily harder to format and read and it takes up more vertical
space.

 src/spice-streaming-agent.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/spice-streaming-agent.cpp b/src/spice-streaming-agent.cpp
index 87e8fa3..ad52277 100644
--- a/src/spice-streaming-agent.cpp
+++ b/src/spice-streaming-agent.cpp
@@ -355,8 +355,8 @@ do_capture(const string &streamport, FILE *f_log)
 
     streamfd = open(streamport.c_str(), O_RDWR);
     if (streamfd < 0)
-        // TODO was syslog(LOG_ERR, "Failed to open %s: %s\n", streamport, strerror(errno));
-        throw std::runtime_error("failed to open streaming device");
+        throw std::runtime_error("failed to open the streaming device (" +
+                                 streamport + "): " + strerror(errno));
 
     unsigned int frame_count = 0;
     while (! quit) {
-- 
2.15.1

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]