Hi Zack,
thanks for this patch. Can you please provide a first line of commit
message in line with:
https://docs.rtems.org/branches/master/eng/vc-users.html#commit-message-guidance
Specifically, the first line of your commit should preferably start
with "shell:" or "libmisc/shell:" and if possible briefly summarize
what is fixed "shell: fixes cut-and-paste in main_edit ..."
Gedare
On Sat, Feb 4, 2023 at 1:24 PM zack leung <zakthertems...@gmail.com> wrote:
>
> fixes #4557
> ---
> cpukit/libmisc/shell/main_edit.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cpukit/libmisc/shell/main_edit.c
> b/cpukit/libmisc/shell/main_edit.c
> index 191eefa19d..71bb1d931b 100644
> --- a/cpukit/libmisc/shell/main_edit.c
> +++ b/cpukit/libmisc/shell/main_edit.c
> @@ -1710,7 +1710,7 @@ static void copy_selection(struct editor *ed) {
> ed->env->clipboard = (unsigned char *) realloc(ed->env->clipboard,
> ed->env->clipsize);
> if (!ed->env->clipboard) return;
> copy(ed, ed->env->clipboard, selstart, ed->env->clipsize);
> - select_toggle(ed);
> +
Avoid adding white space
> }
>
> static void cut_selection(struct editor *ed) {
> @@ -2128,7 +2128,7 @@ static void edit(struct editor *ed) {
>
> case ctrl('e'): select_toggle(ed); break;
> case ctrl('a'): select_all(ed); break;
> - case ctrl('c'): copy_selection(ed); break;
> + case ctrl('c'): copy_selection(ed); select_toggle(ed); break;
> case ctrl('f'): find_text(ed, 0); break;
> case ctrl('l'): goto_line(ed); break;
> case ctrl('g'): find_text(ed, 1); break;
> --
> 2.39.1
> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
