[CSStep] Remove extraneous ) printing after PotentialBindings/Disjunctions.

`debugLogger` moved into each if-block so it's indentation doesn't affect next `isDebugMode`.

Author: Amritpan Kaur

lib/Sema/CSStep.cpp

@@ -358,20 +358,19 @@ StepResult ComponentStep::take(bool prevFailed) {
   auto *disjunction = CS.selectDisjunction();
 
   if (CS.isDebugMode()) {
-    PrintOptions PO;
-    PO.PrintTypesForDebugging = true;
-
-    auto &log = getDebugLogger();
     if (!potentialBindings.empty()) {
+      auto &log = getDebugLogger();
       log << "(Potential Binding(s): " << '\n';
       log << potentialBindings;
     }
-    log.indent(CS.solverState->getCurrentIndent());
 
     SmallVector<Constraint *, 4> disjunctions;
     CS.collectDisjunctions(disjunctions);
     std::vector<std::string> choicesWithBindOverload;
     for (const auto &disjunction : disjunctions) {
+      PrintOptions PO;
+      PO.PrintTypesForDebugging = true;
+
       bool isBindOverload = false;
       auto constraints = disjunction->getNestedConstraints();
       for (const auto &choice : constraints) {
@@ -383,12 +382,17 @@ StepResult ComponentStep::take(bool prevFailed) {
             constraints.front()->getFirstType()->getString(PO));
     }
     if (!choicesWithBindOverload.empty()) {
+      auto &log = getDebugLogger();
       log.indent(2);
       log << "Disjunction(s) = [";
       interleave(choicesWithBindOverload, log, ", ");
-      log << "]";
+      log << "]\n";
+
+      if (!potentialBindings.empty() || !choicesWithBindOverload.empty()) {
+        auto &log = getDebugLogger();
+        log << ")\n";
+      }
     }
-    log << ")\n";
   }
 
   if (CS.shouldAttemptFixes()) {
@@ -484,7 +488,7 @@ StepResult ComponentStep::take(bool prevFailed) {
 
   Solutions.push_back(std::move(solution));
   return finalize(/*isSuccess=*/true);
-}
+  }
 
 StepResult ComponentStep::finalize(bool isSuccess) {
   // If this was a single component, there is nothing to be done,