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

2017-03-15 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL297812: Remove lldb streams from the Log class completely (authored by labath). Changed prior to commit: https://reviews.llvm.org/D30894?vs=91562&id=91841#toc Repository: rL LLVM https://reviews.llv

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

2017-03-15 Thread 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(stder

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

2017-03-14 Thread Zachary Turner via Phabricator via lldb-commits
zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land. Looks great. Comment at: tools/lldb-server/LLDBServerUtilities.cpp:59-60 if (!success) { - fprintf(stderr, "Unable to open log file '%s' for channel \"%s\"\n", -

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

2017-03-13 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. Herald added a subscriber: emaste. previously we switched to llvm streams for log output, this completes the switch for the error streams. I also clean up the includes and remove the unused argument from DisableAllLogChannels(). This required adding a bit of boiler