message: [Target] Cleanup ThreadPlanStepInRange::DefaultShouldStopHereImpl
  This appears to be fallout from the NULL -> nullptr conversion.
diff: @@ -505,9 +505,6 @@ bool ThreadPlanStepInRange::DefaultShouldStopHereImpl(Flags &flags,
  Log *log(lldb_private::GetLogIfAllCategoriesSet(LIBLLDB_LOG_STEP));
  if (m_step_into_target) {
  - SymbolContext sc = frame->GetSymbolContext(
  - eSymbolContextFunction | eSymbolContextBlock | eSymbolContextSymbol);
  - if (sc.symbol != NULL) {
  SymbolContext sc = frame->GetSymbolContext(
  eSymbolContextFunction | eSymbolContextBlock | eSymbolContextSymbol);
  if (sc.symbol != nullptr) {
  @@ -529,19 +526,6 @@ bool ThreadPlanStepInRange::DefaultShouldStopHereImpl(Flags &flags,
  "target %s.",
  sc.GetFunctionName().AsCString(),
  m_step_into_target.AsCString());
  - } else {
  - const char *target_name = m_step_into_target.AsCString();
  - const char *function_name = sc.GetFunctionName().AsCString();
  -
  - if (function_name == NULL)
  - should_step_out = true;
  - else if (strstr(function_name, target_name) == NULL)
  - should_step_out = true;
  - }
  - if (log && should_step_out)
  - log->Printf(
  - "Stepping out of frame %s which did not match step into target %s.",
  - sc.GetFunctionName().AsCString(), m_step_into_target.AsCString());
  }
  }
  