Re: filling long html href

Re: filling long html href

[Richard Stallman]

    I see in fill-move-to-break-point that fill-nobreak-predicate is
    ignored (let bound to nil) if there's no breaks found before
    fill-column.  It seems to me not a good idea to sometimes obey that
    var and sometimes not.

Does anyone recall why that binding was added?

Re: filling long html href

[Stefan Monnier]

>     I see in fill-move-to-break-point that fill-nobreak-predicate is
>     ignored (let bound to nil) if there's no breaks found before
>     fill-column.  It seems to me not a good idea to sometimes obey that
>     var and sometimes not.

> Does anyone recall why that binding was added?

I believe it was just me being confused,


        Stefan

Re: filling long html href

[Chong Yidong]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>     I see in fill-move-to-break-point that fill-nobreak-predicate is
>>     ignored (let bound to nil) if there's no breaks found before
>>     fill-column.  It seems to me not a good idea to sometimes obey that
>>     var and sometimes not.
>
>> Does anyone recall why that binding was added?
>
> I believe it was just me being confused,

OK, I checked it in.  Thanks.

Re: filling long html href

[Chong Yidong]

Chong Yidong <cyd@stupidchicken.com> writes:

> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>>     I see in fill-move-to-break-point that fill-nobreak-predicate is
>>>     ignored (let bound to nil) if there's no breaks found before
>>>     fill-column.  It seems to me not a good idea to sometimes obey that
>>>     var and sometimes not.
>>
>>> Does anyone recall why that binding was added?
>>
>> I believe it was just me being confused,
>
> OK, I checked it in.  Thanks.

Actually, on second thought, this should probably wait till after the
release.  The current code has been in there for > 4 years, without a
complaint till now, and the current bug is more of a nuisance than a
significant problem.

So I reverted that patch for now.

Re: filling long html href

[Stefan Monnier]

>>>> I see in fill-move-to-break-point that fill-nobreak-predicate is
>>>> ignored (let bound to nil) if there's no breaks found before
>>>> fill-column.  It seems to me not a good idea to sometimes obey that
>>>> var and sometimes not.
>>> 
>>>> Does anyone recall why that binding was added?
>>> 
>>> I believe it was just me being confused,
>> 
>> OK, I checked it in.  Thanks.

> Actually, on second thought, this should probably wait till after the
> release.  The current code has been in there for > 4 years, without a
> complaint till now, and the current bug is more of a nuisance than a
> significant problem.

But this is new behavior compared to Emacs-21.4.  It's not a significant
problem, but it's not an improvement either.

> So I reverted that patch for now.

I installed it (and I haven't seen where/when you'd installed yours: doesn't
appear in the ChangeLog).


        Stefan

Re: filling long html href

[Chong Yidong]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>>>> I see in fill-move-to-break-point that fill-nobreak-predicate is
>>>>> ignored (let bound to nil) if there's no breaks found before
>>>>> fill-column.  It seems to me not a good idea to sometimes obey that
>>>>> var and sometimes not.
>>>> 
>>>>> Does anyone recall why that binding was added?
>>>> 
>>>> I believe it was just me being confused,
>>> 
>>> OK, I checked it in.  Thanks.
>
>> Actually, on second thought, this should probably wait till after the
>> release.  The current code has been in there for > 4 years, without a
>> complaint till now, and the current bug is more of a nuisance than a
>> significant problem.
>
> But this is new behavior compared to Emacs-21.4.  It's not a significant
> problem, but it's not an improvement either.
>
>> So I reverted that patch for now.
>
> I installed it (and I haven't seen where/when you'd installed yours: doesn't
> appear in the ChangeLog).

OK.