[Lldb-commits] [PATCH] Add -stack-info-frame command (MI)

Ed Maste emaste at freebsd.org
Thu Feb 19 06:21:37 PST 2015


================
Comment at: tools/lldb-mi/MICmdCmdStack.h:80
@@ +79,3 @@
+// Gotchas: None.
+// Authors: Ilia Kirianovskii 23/01/2015.
+// Changes: None.
----------------
abidh wrote:
> I think we should not continue this practice of adding the name of the authors. 
The `Gotchas` and `Changes` header entries are also atypical for LLDB.

Looking through tools/lldb-mi I see exactly one `Changes` entry that is not 'None`:
```
MICmnStreamStdin.h:38:// Changes: Factored out OS specific handling of reading stdin  - IOR 16/06/2014.
```
SVN is very good at tracking changes to these files; there's no need to try to do it manually as well.

I also see only four `Gotchas` that are not `None`, two of which are the same. I'd suggest we just avoid these in newly added code.

http://reviews.llvm.org/D7750

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list