bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#54804: 29.0.50; zap-to-char: case sensitive for upper-case letter


From: Tino Calancha
Subject: bug#54804: 29.0.50; zap-to-char: case sensitive for upper-case letter
Date: Wed, 18 May 2022 09:25:46 +0200

Eli Zaretskii <eliz@gnu.org> writes:

>> diff --git a/test/lisp/simple-tests.el b/test/lisp/simple-tests.el
>> index dcab811bb5..2baf7124d7 100644
>> --- a/test/lisp/simple-tests.el
>> +++ b/test/lisp/simple-tests.el
>> @@ -971,5 +971,27 @@ test-undo-region
>>      ;;(should (= (length (delq nil (undo-make-selective-list 5 9))) 0))
>>      (should (= (length (delq nil (undo-make-selective-list 6 9))) 0))))
>>  
>> +
>> +;;; Tests for `zap-to-char'
>
> Should we have the tests for zap-* functions in one place, rather than
> two?

My understanding is that we only test in `foo-tests.el' functions
beloging to `foo.el'.
For historical reasons:
`zap-to-char' is defined in simple.el
`zap-up-to-char' is defined in misc.el

We can move the latter to simple.el, then test the functions in the same file.
Some people dislike that because it affects the Git history.
I am OK with whatever the people prefer here.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]