cutting: cover the corner cases where cut commands do not cut anything

Give the correct feedback in those cases and do not add an undo item.

This fixes both parts of https://savannah.gnu.org/bugs/?55396.
master
Benno Schulenberg 2019-01-06 11:56:42 +01:00
parent 50e8f8aab9
commit 67873e961b
2 changed files with 18 additions and 9 deletions

View File

@ -203,7 +203,8 @@ void do_cut_prev_word(void)
/* Delete a word rightward. */ /* Delete a word rightward. */
void do_cut_next_word(void) void do_cut_next_word(void)
{ {
do_cutword(FALSE); if (!nothing_needs_cutting(openfile->current_x > 0))
do_cutword(FALSE);
} }
#endif /* !NANO_TINY */ #endif /* !NANO_TINY */
@ -356,14 +357,18 @@ void do_cut_text(bool copy_text, bool marked, bool cut_till_eof, bool append)
} }
/* Return TRUE when a cut command would not actually cut anything: when /* Return TRUE when a cut command would not actually cut anything: when
* on an empty line at EOF, or when the mark covers zero characters. */ * on an empty line at EOF, or when the mark covers zero characters, or
bool nothing_needs_cutting(void) * (when test_cliff is TRUE) when the magic line would be cut. */
bool nothing_needs_cutting(bool test_cliff)
{ {
if ((openfile->current->next == NULL && openfile->current->data[0] == '\0' if ((openfile->current->next == NULL && openfile->current->data[0] == '\0'
#ifndef NANO_TINY #ifndef NANO_TINY
&& openfile->mark == NULL) || && openfile->mark == NULL) ||
(openfile->mark == openfile->current && (openfile->mark == openfile->current &&
openfile->mark_x == openfile->current_x openfile->mark_x == openfile->current_x) ||
(test_cliff && openfile->current->data[openfile->current_x] == '\0' &&
((ISSET(NO_NEWLINES) && openfile->current == openfile->filebot) ||
(!ISSET(NO_NEWLINES) && openfile->current == openfile->filebot->prev))
#endif #endif
)) { )) {
#ifndef NANO_TINY #ifndef NANO_TINY
@ -378,10 +383,10 @@ bool nothing_needs_cutting(void)
/* Move text from the current buffer into the cutbuffer. */ /* Move text from the current buffer into the cutbuffer. */
void do_cut_text_void(void) void do_cut_text_void(void)
{ {
if (nothing_needs_cutting()) #ifndef NANO_TINY
if (nothing_needs_cutting(ISSET(CUT_FROM_CURSOR) && openfile->mark == NULL))
return; return;
#ifndef NANO_TINY
/* Only add a new undo item when the current item is not a CUT or when /* Only add a new undo item when the current item is not a CUT or when
* the current cut is not contiguous with the previous cutting. */ * the current cut is not contiguous with the previous cutting. */
if (openfile->last_action != CUT || openfile->current_undo == NULL || if (openfile->last_action != CUT || openfile->current_undo == NULL ||
@ -391,7 +396,8 @@ void do_cut_text_void(void)
do_cut_text(FALSE, openfile->mark, FALSE, FALSE); do_cut_text(FALSE, openfile->mark, FALSE, FALSE);
update_undo(CUT); update_undo(CUT);
#else #else
do_cut_text(FALSE, FALSE, FALSE, FALSE); if (!nothing_needs_cutting(FALSE))
do_cut_text(FALSE, FALSE, FALSE, FALSE);
#endif #endif
} }
@ -430,7 +436,9 @@ void do_copy_text(void)
/* Cut from the current cursor position to the end of the file. */ /* Cut from the current cursor position to the end of the file. */
void do_cut_till_eof(void) void do_cut_till_eof(void)
{ {
if (openfile->current->next == NULL && openfile->current->data[0] == '\0') { if ((openfile->current == openfile->filebot && openfile->current->data[0] == '\0') ||
(!ISSET(NO_NEWLINES) && openfile->current->next == openfile->filebot &&
openfile->current->data[openfile->current_x] == '\0')) {
statusbar(_("Nothing was cut")); statusbar(_("Nothing was cut"));
return; return;
} }
@ -447,7 +455,7 @@ void zap_text(void)
filestruct *was_cutbuffer = cutbuffer; filestruct *was_cutbuffer = cutbuffer;
filestruct *was_cutbottom = cutbottom; filestruct *was_cutbottom = cutbottom;
if (nothing_needs_cutting()) if (nothing_needs_cutting(ISSET(CUT_FROM_CURSOR) && openfile->mark == NULL))
return; return;
/* Add a new undo item only when the current item is not a ZAP or when /* Add a new undo item only when the current item is not a ZAP or when

View File

@ -251,6 +251,7 @@ void do_cut_next_word(void);
void cut_marked(bool *right_side_up); void cut_marked(bool *right_side_up);
#endif #endif
void do_cut_text(bool copy_text, bool marked, bool cut_till_eof, bool append); void do_cut_text(bool copy_text, bool marked, bool cut_till_eof, bool append);
bool nothing_needs_cutting(bool test_cliff);
void do_cut_text_void(void); void do_cut_text_void(void);
#ifndef NANO_TINY #ifndef NANO_TINY
void do_copy_text(void); void do_copy_text(void);