leonid.mashinskiy added inline comments.

================
Comment at: source/Expression/DWARFExpression.cpp:3253
 
-  return false;
+  return true;
 }
----------------
zturner wrote:
> Why do we change the return value here?
This function had mixed return value semantics and I changed it to always 
returning true on success and false on failure.
Nobody checked return value before and I use it in unit tests.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55122/new/

https://reviews.llvm.org/D55122



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to