[Lldb-commits] [PATCH] Initialize ProcessPOSIXLog by NativeProcessLinux

Tamas Berghammer tberghammer at google.com
Thu Mar 5 08:59:46 PST 2015


We can, but ProcessPOSIXLog is a dependency for both of them, so I think it isn't a good idea.

If we want to reduce the amount of code duplication, then I would suggest to simplify the process of registering a log channel to just one function call (e.g.: create a ProcessPOSIXLog::RegisterLogger (const ConstString& name)) function.


http://reviews.llvm.org/D8080

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list