[Lldb-commits] [PATCH] D30894: Remove lldb streams from the Log class completely

[Pavel Labath via Phabricator via lldb-commits]

labath added inline comments.


================
Comment at: tools/lldb-server/LLDBServerUtilities.cpp:59-60
     if (!success) {
-      fprintf(stderr, "Unable to open log file '%s' for channel \"%s\"\n",
-              log_file.c_str(), channel_with_categories.str().c_str());
+      fprintf(stderr, "Unable to setup logging for channel \"%s\": %s",
+              channel.c_str(), error_stream.str().c_str());
       return false;
----------------
zturner wrote:
> How about `llvm::errs() << formatv("Unable to setup logging for channel \"{0}\": {1}\n", channel, error_stream.str());`
good point. will do.


https://reviews.llvm.org/D30894