[Lldb-commits] [PATCH] Update the LLDB website with more information

Jim Ingham jingham at apple.com
Tue Mar 10 12:20:35 PDT 2015


Couple of comments.

First off, if you are building with Xcode, you don't need to check out llvm & clang, the build will do that for you if they don't already exist.  So for people who want to work on lldb & don't have an extant llvm/clang checkout, it is easier to just check out lldb & build.

By saying "The line limit IS 120 characters"... "However you will see places" you make it sound like the exceptions are of questionable status, which is not at all the intent.  Particularly for these tables, letting them run longer is IMO preferable (there are a few of the tables where the help text wraps & I find those harder to read.)  You could say something like:

The line limit for code statements is...

Then it would be clearer that this statement does not contradict the immediately following paragraph.


http://reviews.llvm.org/D8222

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






More information about the lldb-commits mailing list