We set the error when a breakpoint condition

doesn't return anything; that's great.

We should probably also return rather than
trying to access the nonexistent return value.

<rdar://problem/14009519>

llvm-svn: 184765

diff --git a/lldb/source/Breakpoint/BreakpointLocation.cpp b/lldb/source/Breakpoint/BreakpointLocation.cpp
index ba8f1de8d12806372e3bb33fbf9d2c269285ca66..b32937b6445bb74dff1f2ae2deccff69b40445ff 100644 (file)
--- a/lldb/source/Breakpoint/BreakpointLocation.cpp
+++ b/lldb/source/Breakpoint/BreakpointLocation.cpp
@@ -319,6 +319,7 @@ BreakpointLocation::ConditionSaysStop (ExecutionContext &exe_ctx, Error &error)
         {
             ret = false;
             error.SetErrorString("Expression did not return a result");
+            return false;
         }
         
         result_value_sp = result_variable_sp->GetValueObject();