More review comments.

Author: jimingham

lldb/source/Breakpoint/BreakpointSite.cpp

@@ -92,12 +92,13 @@ std::optional<uint32_t> BreakpointSite::GetSuggestedStackFrameIndex() {
   std::optional<uint32_t> result;
   std::lock_guard<std::recursive_mutex> guard(m_constituents_mutex);
   for (BreakpointLocationSP loc_sp : m_constituents.BreakpointLocations()) {
-    std::optional<uint32_t> this_result = loc_sp->GetSuggestedStackFrameIndex();
-    if (this_result) {
-      if (!result)
-        result = this_result;
+    std::optional<uint32_t> loc_frame_index 
+        = loc_sp->GetSuggestedStackFrameIndex();
+    if (loc_frame_index) {
+      if (result)
+        result = std::max(*loc_frame_index, *result);
       else
-        result = std::max(*this_result, *result);
+        result = this_result;
     }
   }
   return result;