[Lldb-commits] [PATCH] D11052: [lldb-mi] Simplify MICmnMIResultRecord implementation.

[Ilia K]

ki.stfu added a comment.

Looks good, but how about returning NULL before exiting from MapResultClassToResultClassText?


================
Comment at: tools/lldb-mi/MICmnMIResultRecord.cpp:35
@@ +34,3 @@
+    }
+}
+
----------------
here:
```
  assert(0 && "unknown CMICmnMIResultRecord::ResultClass_e");
  return NULL;
```


http://reviews.llvm.org/D11052