[Lldb-commits] [lldb] c77fc00 - [lldb/Docs] Remove the version from the home page

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 2 14:48:45 PST 2020


If you feel strongly about this and want to change it to zero, be my guest.
:-)

I personally don't see any value in an additional comment.

On Mon, Mar 2, 2020 at 2:43 PM Davide Italiano <ditaliano at apple.com> wrote:

>
>
> On Mar 2, 2020, at 14:35, Jonas Devlieghere <jonas at devlieghere.com> wrote:
>
>
>
> On Mon, Mar 2, 2020 at 2:28 PM Davide Italiano <ditaliano at apple.com>
> wrote:
>
>> You might want to consider removing it entirely instead of commenting it,
>> unless you find some value in keeping something that’s already stale around
>> for the future (e.g. re-enabling).
>>
>
> If you take a look at the rest of the file you'll see it's consistent with
> other options that are disabled. I believe this is the Sphinx default. It
> also communicates that this is disabled on purpose and not just an
> oversight or option added in a later version.
>
>
>
> Then you might consider setting the version to 0 or to the current
> version. And adding a comment explaining this has no semantic value
> associated.
> Leaving a stale version is confusing for somebody who opens the file, e.g.
> me.
>
>> Davide
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200302/093e2704/attachment-0001.html>


More information about the lldb-commits mailing list