Re: adaptive-fill bug?

2005-06-13 Thread Richard Stallman
AFAIK, the check is there to make sure we do not use a fill-prefix which matches paragraph-start. The late-check ensures this without caring where the guessed prefix comes from, so it covers adaptive-fill-regexp matches on both the first and second line (and third line for people

Re: adaptive-fill bug?

2005-06-13 Thread Stefan Monnier
AFAIK, the check is there to make sure we do not use a fill-prefix which matches paragraph-start. The late-check ensures this without caring where the guessed prefix comes from, so it covers adaptive-fill-regexp matches on both the first and second line (and third line

Re: adaptive-fill bug?

2005-06-13 Thread Katsumi Yamaoka
In [EMAIL PROTECTED] Katsumi Yamaoka wrote: I did the overstepping but I'll withdraw the changes in Gnus and SEMI. I've withdrawn them. Thanks. ___ Emacs-pretest-bug mailing list Emacs-pretest-bug@gnu.org

Re: adaptive-fill bug?

2005-06-12 Thread Stefan Monnier
2. it rejects fill-prefixes guessed by adaptive-fill-mode if they are longer than a line of text (longer than the difference between fill-column and left-margin). The second part is the crucial one and should be 100% uncontroversial. I agree. The first part should be

Re: adaptive-fill bug?

2005-06-11 Thread Richard Stallman
2. it rejects fill-prefixes guessed by adaptive-fill-mode if they are longer than a line of text (longer than the difference between fill-column and left-margin). The second part is the crucial one and should be 100% uncontroversial. I agree. The first part should be

Re: adaptive-fill bug?

2005-06-10 Thread Richard Stallman
Could you explain the overall idea of this patch? Changes in the heuristics of adaptive-fill-mode are very risky. Please don't go ahead and install it before we have this discussion. ___ Emacs-pretest-bug mailing list Emacs-pretest-bug@gnu.org

Re: adaptive-fill bug?

2005-06-10 Thread Stefan Monnier
Could you explain the overall idea of this patch? The patch makes two changes: 1. it removes the (looking-at paragraph-start) check on the second line of a paragraph. 2. it rejects fill-prefixes guessed by adaptive-fill-mode if they are longer than a line of text (longer than the

Re: adaptive-fill bug?

2005-06-09 Thread Katsumi Yamaoka
In [emacs-w3m : No.08153] Katsumi Yamaoka wrote: (with-temp-buffer (insert-char ?- 70) (let ((fill-column 70) (adaptive-fill-mode t)) (fill-region (point-min) (point-max = fill-prefix too long for specified width Is it reasonable that the form causes an error? Similar

Re: adaptive-fill bug?

2005-06-09 Thread Stefan Monnier
While I'm not sure of it since I don't use the adaptive-fill-mode normally, I found something like a bug. Try the following: (with-temp-buffer (insert-char ?- 70) (let ((fill-column 70) (adaptive-fill-mode t)) (fill-region (point-min) (point-max = fill-prefix too long

Re: adaptive-fill bug?

2005-06-09 Thread Katsumi Yamaoka
In [emacs-w3m : No.08172] Stefan Monnier wrote: Does the patch below fix your problem? Very good. It seems to be a right course not to look back on paragraph-start in that situation. Furthermore, ignoring long prefix completes it. Could you please install it? Though we can solve it for

adaptive-fill bug?

2005-06-07 Thread Katsumi Yamaoka
Hi, While I'm not sure of it since I don't use the adaptive-fill-mode normally, I found something like a bug. Try the following: (with-temp-buffer (insert-char ?- 70) (let ((fill-column 70) (adaptive-fill-mode t)) (fill-region (point-min) (point-max = fill-prefix too long

Re: adaptive-fill bug?

2005-06-07 Thread Katsumi Yamaoka
In [emacs-w3m : No.08153] Katsumi Yamaoka wrote: (with-temp-buffer (insert-char ?- 70) (let ((fill-column 70) (adaptive-fill-mode t)) (fill-region (point-min) (point-max = fill-prefix too long for specified width No problem in Emacs 21. The cause seems to be the