[Lldb-commits] [PATCH] D29615: Convert Log class to llvm streams

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 6 19:10:59 PST 2017


zturner added a comment.

Where is the logging callback used?  It seems rather odd that we would need to notify someone every time a log message was generated.



================
Comment at: source/Core/StreamCallback.cpp:22
 StreamCallback::StreamCallback(lldb::LogOutputCallback callback, void *baton)
-    : Stream(0, 4, eByteOrderBig), m_callback(callback), m_baton(baton),
-      m_accumulated_data(), m_collection_mutex() {}
----------------
I find it rather odd that this was hardcoding big endian.  Was the endianness here important for some reason?


https://reviews.llvm.org/D29615





More information about the lldb-commits mailing list