[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-
From: |
Po Lu |
Subject: |
Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug' |
Date: |
Thu, 21 Jul 2022 21:56:43 +0800 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.91 (gnu/linux) |
Gregory Heytings <gregory@heytings.org> writes:
> Not a boolean, because even if there is no reason for users to change
> the number, there are at least three useful values: always disabled,
> enabled iff the buffer contains long lines, always enabled. Which you
> can get now by setting the variable to nil, 10000, and 0.
Then why isn't that written in the doc string? And with more obvious
values: nil, t, always.
> I'm also pretty sure we would have bug reports if what a "long line"
> is was hardcoded. So from my point of view the variable is fine as it
> is.
Right, so users will complain about a constant in a supposedly
transparent feature? Then it's not as transparent as you think it is.
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', (continued)
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Po Lu, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Po Lu, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Po Lu, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Eli Zaretskii, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Eli Zaretskii, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug',
Po Lu <=
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Eli Zaretskii, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Eli Zaretskii, 2022/07/21
- Re: master 616da8fa8e: Merge branch 'feature/fix-the-long-lines-display-bug', Gregory Heytings, 2022/07/21