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

[Zachary Turner]

I wasn't aware that the Xcode build did that. I'll update in a bit. I think
it's useful to say something about the directory structure though because
it lets people know how to update their code. Also will fix the suggestions
about line length
On Tue, Mar 10, 2015 at 12:20 PM Jim Ingham <jingham at apple.com> wrote:

> 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/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150310/1dbd3edc/attachment.html>