clayborg accepted this revision.
clayborg added a comment.

So I am going to let this through because you have done all we asked and I 
really like the "source info" command!

But in the future when adding things for MI, please add an API in the lldb::SB 
layer to do what you need. Text scraping is really not the solution we should 
be adding into the code. Other people using MI are going to see this kind of 
code and think it is what they should do when implementing/fixing new/existing 
MI commands.

We have an API for a debugger that we are trying to make with the lldb::SB 
layer, so we should use it and stop ANY form of text scraping in the future. I 
think an API'ized version of what you are requesting here -- all line table 
entries for a given source file -- should be available from SBTarget and a 
future modification to MICmdCmdSymbol.cpp should switch over to using it.


Repository:
  rL LLVM

http://reviews.llvm.org/D15593



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

Reply via email to