clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

Close, we just need to remove the "Headers for" string. See inlined comment.


================
Comment at: source/Commands/CommandObjectTarget.cpp:1568
@@ +1567,3 @@
+                ObjectFile *objfile = module->GetObjectFile();
+                strm.Printf("Headers for '%s':\n", 
objfile->GetFileSpec().GetCString());
+                strm.IndentMore();
----------------
We should probably leave this for the objfile->Dump(...) to print.  "Headers 
for '%s'" doesn't make sense here. So this print should probably be removed, or 
just the filename should be printed out without the "Headers for" part.


http://reviews.llvm.org/D18464



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

Reply via email to