unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#47505: 25.1; ediff thinks words changed when filling prefixed
@ 2021-03-30 22:02 ` Samuel Wales
  2021-03-31  7:25   ` Andreas Schwab
       [not found]   ` <handler.47505.C.162047678518512.notifdonectrl.0@debbugs.gnu.org>
  0 siblings, 2 replies; 10+ messages in thread
From: Samuel Wales @ 2021-03-30 22:02 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 47505

to reproduce, please put this in file a:

# asdfjkansdkljfan ksd fnak nsdflkadsnfkansdkfnakdsnf
# aklsfdnakljfnsdkasdnfja sdn fkljad nsfkj ansdkfn aksdjfna
# ksjfndk ajdnsf kajsndkfjansdkfjn ja nsdkjfn askdj fnakjs fdnakj
# nsdfkaj dns

then set fill-column to 60.  then fill.  then save to file b.

you now have a and b with the same text, but a filled commented
paragraph that differs between them.

now open in diff mode.  you will notice that it thinks only the comments
changed.  this is expected behavior.

now open with m-x ediff RET.  it thinks the first word in the last line
in b is different.  but it is not different.

best i can do for a bug report.  thanks.


On 3/28/21, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: Samuel Wales <samologist@gmail.com>
>> Date: Sun, 28 Mar 2021 21:48:18 -0700
>> Cc: help-gnu-emacs@gnu.org
>>
>> for purpoes of the test do something like this:
>>
>> # asdfjkansdkljfan ksd fnak nsdflkadsnfkansdkfnakdsnf
>> # aklsfdnakljfnsdkasdnfja sdn fkljad nsfkj ansdkfn aksdjfna
>> # ksjfndk ajdnsf kajsndkfjansdkfjn ja nsdkjfn askdj fnakjs fdnakj
>> # nsdfkaj dns
>>
>> then fill to a narrow fill column.
>
> Please submit a full bug report with the details, TIA.
>
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-30 22:02 ` bug#47505: 25.1; ediff thinks words changed when filling prefixed Samuel Wales
@ 2021-03-31  7:25   ` Andreas Schwab
  2021-03-31 19:28     ` Samuel Wales
  2021-03-31 19:30     ` Samuel Wales
       [not found]   ` <handler.47505.C.162047678518512.notifdonectrl.0@debbugs.gnu.org>
  1 sibling, 2 replies; 10+ messages in thread
From: Andreas Schwab @ 2021-03-31  7:25 UTC (permalink / raw)
  To: Samuel Wales; +Cc: 47505

On Mär 30 2021, Samuel Wales wrote:

> now open with m-x ediff RET.  it thinks the first word in the last line
> in b is different.  but it is not different.

I don't see how that is a bug.  The word moved around, and ediff
correctly marked it as such.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-31  7:25   ` Andreas Schwab
@ 2021-03-31 19:28     ` Samuel Wales
  2021-03-31 23:53       ` Michael Heerdegen
  2021-03-31 19:30     ` Samuel Wales
  1 sibling, 1 reply; 10+ messages in thread
From: Samuel Wales @ 2021-03-31 19:28 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 47505

doyou think diff's behavior is a bug then?  i choose ediff's.

On 3/31/21, Andreas Schwab <schwab@linux-m68k.org> wrote:
> On Mär 30 2021, Samuel Wales wrote:
>
>> now open with m-x ediff RET.  it thinks the first word in the last line
>> in b is different.  but it is not different.
>
> I don't see how that is a bug.  The word moved around, and ediff
> correctly marked it as such.
>
> Andreas.
>
> --
> Andreas Schwab, schwab@linux-m68k.org
> GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
> "And now for something completely different."
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-31  7:25   ` Andreas Schwab
  2021-03-31 19:28     ` Samuel Wales
@ 2021-03-31 19:30     ` Samuel Wales
  1 sibling, 0 replies; 10+ messages in thread
From: Samuel Wales @ 2021-03-31 19:30 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 47505

do you think diff-mode's behavior is a bug?

i would say that ediff's is a bug because moving around is not what
seems significant.  comments should be honorary whitespace.


On 3/31/21, Andreas Schwab <schwab@linux-m68k.org> wrote:
> On Mär 30 2021, Samuel Wales wrote:
>
>> now open with m-x ediff RET.  it thinks the first word in the last line
>> in b is different.  but it is not different.
>
> I don't see how that is a bug.  The word moved around, and ediff
> correctly marked it as such.
>
> Andreas.
>
> --
> Andreas Schwab, schwab@linux-m68k.org
> GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
> "And now for something completely different."
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-31 19:28     ` Samuel Wales
@ 2021-03-31 23:53       ` Michael Heerdegen
  2021-04-01  0:04         ` Samuel Wales
  2021-05-08 12:26         ` Lars Ingebrigtsen
  0 siblings, 2 replies; 10+ messages in thread
From: Michael Heerdegen @ 2021-03-31 23:53 UTC (permalink / raw)
  To: Samuel Wales; +Cc: Andreas Schwab, 47505

Samuel Wales <samologist@gmail.com> writes:

> doyou think diff's behavior is a bug then?  i choose ediff's.

Depends on what you think what "fine diffs" are.  They are actually word
wise diffs, and comment starts are a just a word.  That how it's
implemented.  In the case you describe I find that sometimes annoying,
too.  But I guess it's not easy to change that.

It might be possible to tell Ediff to ignore certain words (like comment
starts) but it's then not trivial to decide whether some text got
commented that was not commented before.

This is all not trivial.  There can be lines that are partly code and
partly comment; in Elisp, the comment start can have changed (e.g. ;; ->
;;;), you can have "doubled comments" like ;;; ;;; where uncommenting
would leave comments ... how would you define when two regions
should count as similar?

Regards,

Michael.





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-31 23:53       ` Michael Heerdegen
@ 2021-04-01  0:04         ` Samuel Wales
  2021-04-01  0:17           ` Michael Heerdegen
  2021-05-08 12:26         ` Lars Ingebrigtsen
  1 sibling, 1 reply; 10+ messages in thread
From: Samuel Wales @ 2021-04-01  0:04 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: Andreas Schwab, 47505

i don't know.  i just know what seems right and seems wrong.  not too
useful is it?  but it is better than not filing the bug perhaps.

diff-mode does it right so ediff can.  filladapt can intuit prefixes.
comment syntaxes are known.  somewhere in there lies a fix maybe.

commented paragraphs in org/shell/elisp style are always commented at
bol.  idk what diff-mode does.


On 3/31/21, Michael Heerdegen <michael_heerdegen@web.de> wrote:
> Samuel Wales <samologist@gmail.com> writes:
>
>> doyou think diff's behavior is a bug then?  i choose ediff's.
>
> Depends on what you think what "fine diffs" are.  They are actually word
> wise diffs, and comment starts are a just a word.  That how it's
> implemented.  In the case you describe I find that sometimes annoying,
> too.  But I guess it's not easy to change that.
>
> It might be possible to tell Ediff to ignore certain words (like comment
> starts) but it's then not trivial to decide whether some text got
> commented that was not commented before.
>
> This is all not trivial.  There can be lines that are partly code and
> partly comment; in Elisp, the comment start can have changed (e.g. ;; ->
> ;;;), you can have "doubled comments" like ;;; ;;; where uncommenting
> would leave comments ... how would you define when two regions
> should count as similar?
>
> Regards,
>
> Michael.
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-04-01  0:04         ` Samuel Wales
@ 2021-04-01  0:17           ` Michael Heerdegen
  0 siblings, 0 replies; 10+ messages in thread
From: Michael Heerdegen @ 2021-04-01  0:17 UTC (permalink / raw)
  To: Samuel Wales; +Cc: Andreas Schwab, 47505

Samuel Wales <samologist@gmail.com> writes:

> i don't know.  i just know what seems right and seems wrong.  not too
> useful is it?  but it is better than not filing the bug perhaps.
>
> diff-mode does it right so ediff can.  filladapt can intuit prefixes.
> comment syntaxes are known.  somewhere in there lies a fix maybe.
>
> commented paragraphs in org/shell/elisp style are always commented at
> bol.  idk what diff-mode does.

I don't know how diff-mode does it, but I wonder if it then doesn't sort
out changes that are significant.  "I don't know" is not meant ironic,
maybe someone else can help further.

Regards,

Michael.





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-03-31 23:53       ` Michael Heerdegen
  2021-04-01  0:04         ` Samuel Wales
@ 2021-05-08 12:26         ` Lars Ingebrigtsen
  2021-05-08 20:42           ` Samuel Wales
  1 sibling, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2021-05-08 12:26 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: Andreas Schwab, Samuel Wales, 47505

Michael Heerdegen <michael_heerdegen@web.de> writes:

> Depends on what you think what "fine diffs" are.  They are actually word
> wise diffs, and comment starts are a just a word.  That how it's
> implemented.  In the case you describe I find that sometimes annoying,
> too.  But I guess it's not easy to change that.

I think this is basically working as intended (even if it's different in
the two cases, because the interfaces emphasise different things).

So I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: acknowledged by developer (control message for bug #47505)
       [not found]   ` <handler.47505.C.162047678518512.notifdonectrl.0@debbugs.gnu.org>
@ 2021-05-08 20:40     ` Samuel Wales
  0 siblings, 0 replies; 10+ messages in thread
From: Samuel Wales @ 2021-05-08 20:40 UTC (permalink / raw)
  To: 47505

thanks for considering it.  if anybody has a fix for this, even as a
hook value, please let me know.  i might incorporate it into something
useful.


On 5/8/21, GNU bug Tracking System <help-debbugs@gnu.org> wrote:
> This is an automatic notification regarding your bug report
> #47505: 25.1; ediff thinks words changed when filling prefixed,
> which was filed against the emacs package.
>
> Thank you for your report, which has now been closed.
> You can view the full report at
> http://debbugs.gnu.org/cgi/bugreport.cgi?bug=47505
>
> If you require further information, please followup to
> 47505@debbugs.gnu.org.
>
> debbugs.gnu.org maintainers
> (administrator, GNU bugs database)
>
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#47505: 25.1; ediff thinks words changed when filling prefixed
  2021-05-08 12:26         ` Lars Ingebrigtsen
@ 2021-05-08 20:42           ` Samuel Wales
  0 siblings, 0 replies; 10+ messages in thread
From: Samuel Wales @ 2021-05-08 20:42 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: Michael Heerdegen, Andreas Schwab, 47505

i don't understand what you mean by the interfaces emphasize different
things.  to me, they should have the same goal here.  in any case,
thanks fort considering it.  if anybody has a fix, please let me know.
i might incorporate it into something useful.

On 5/8/21, Lars Ingebrigtsen <larsi@gnus.org> wrote:
> Michael Heerdegen <michael_heerdegen@web.de> writes:
>
>> Depends on what you think what "fine diffs" are.  They are actually word
>> wise diffs, and comment starts are a just a word.  That how it's
>> implemented.  In the case you describe I find that sometimes annoying,
>> too.  But I guess it's not easy to change that.
>
> I think this is basically working as intended (even if it's different in
> the two cases, because the interfaces emphasise different things).
>
> So I'm closing this bug report.
>
> --
> (domestic pets only, the antidote for overdose, milk.)
>    bloggy blog: http://lars.ingebrigtsen.no
>


-- 
The Kafka Pandemic

Please learn what misopathy is.
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html





^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-05-08 20:42 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <87a6p5qumw.fsf@gnus.org>
2021-03-30 22:02 ` bug#47505: 25.1; ediff thinks words changed when filling prefixed Samuel Wales
2021-03-31  7:25   ` Andreas Schwab
2021-03-31 19:28     ` Samuel Wales
2021-03-31 23:53       ` Michael Heerdegen
2021-04-01  0:04         ` Samuel Wales
2021-04-01  0:17           ` Michael Heerdegen
2021-05-08 12:26         ` Lars Ingebrigtsen
2021-05-08 20:42           ` Samuel Wales
2021-03-31 19:30     ` Samuel Wales
     [not found]   ` <handler.47505.C.162047678518512.notifdonectrl.0@debbugs.gnu.org>
2021-05-08 20:40     ` bug#47505: acknowledged by developer (control message for bug #47505) Samuel Wales

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).