abidh accepted this revision.
abidh added a comment.

Looks good.



================
Comment at: packages/Python/lldbsuite/test/tools/lldb-mi/main.cpp:22
 {
-    int a  = 10;
+    int a = 10;
+
----------------
This declaration looks redundant.


================
Comment at: tools/lldb-mi/MICmdCmdMiscellanous.cpp:515
+      return MIstatus::failure;
   }
 
----------------
aetf wrote:
> abidh wrote:
> > It is not really an OutofBand record but rather an output of the command. 
> > Why not simple add prepend an ~
> The output of the command should be a Stream record which is an OutofBand 
> record according to the spec [1]. I agree it's no more than prepending '~' 
> and quoting the string. But why not just do what the spec says? ;-)
> 
> [1] 
> https://sourceware.org/gdb/onlinedocs/gdb/GDB_002fMI-Stream-Records.html#GDB_002fMI-Stream-Records
ok.


https://reviews.llvm.org/D24711



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to