# HG changeset patch # User Bram Moolenaar # Date 1669248904 -3600 # Node ID b1c66bff0a66a81dd88fa9f63e254c8c61f90be9 # Parent a7801222c9c570bc413b6782e5aaa2caaa6cec44 patch 9.0.0934: various code formatting issues Commit: https://github.com/vim/vim/commit/24fe33a83a5130a5369f06d88000a3a0590a59ec Author: Bram Moolenaar Date: Thu Nov 24 00:09:02 2022 +0000 patch 9.0.0934: various code formatting issues Problem: Various code formatting issues. Solution: Improve code formatting. diff --git a/src/arglist.c b/src/arglist.c --- a/src/arglist.c +++ b/src/arglist.c @@ -413,8 +413,7 @@ arglist_del_files(garray_T *alist_ga) didone = FALSE; for (match = 0; match < ARGCOUNT; ++match) - if (vim_regexec(®match, alist_name(&ARGLIST[match]), - (colnr_T)0)) + if (vim_regexec(®match, alist_name(&ARGLIST[match]), (colnr_T)0)) { didone = TRUE; vim_free(ARGLIST[match].ae_fname); diff --git a/src/gui_xim.c b/src/gui_xim.c --- a/src/gui_xim.c +++ b/src/gui_xim.c @@ -133,7 +133,7 @@ free_xim_stuff(void) #if defined(FEAT_EVAL) || defined(PROTO) /* - * Mark the global 'imactivatefunc' and 'imstatusfunc' callbacks with 'copyID' + * Mark the global 'imactivatefunc' and 'imstatusfunc' callbacks with "copyID" * so that they are not garbage collected. */ int diff --git a/src/libvterm/src/parser.c b/src/libvterm/src/parser.c --- a/src/libvterm/src/parser.c +++ b/src/libvterm/src/parser.c @@ -34,7 +34,7 @@ static void do_csi(VTerm *vt, char comma if(vt->parser.callbacks && vt->parser.callbacks->csi) if((*vt->parser.callbacks->csi)( - vt->parser.v.csi.leaderlen ? vt->parser.v.csi.leader : NULL, + vt->parser.v.csi.leaderlen ? vt->parser.v.csi.leader : NULL, vt->parser.v.csi.args, vt->parser.v.csi.argi, vt->parser.intermedlen ? vt->parser.intermed : NULL, diff --git a/src/list.c b/src/list.c --- a/src/list.c +++ b/src/list.c @@ -109,6 +109,8 @@ list_alloc_id(alloc_id_T id UNUSED) /* * Allocate space for a list, plus "count" items. + * This uses one allocation for efficiency. + * The reference count is not set. * Next list_set_item() must be called for each item. */ list_T * @@ -117,33 +119,34 @@ list_alloc_with_items(int count) list_T *l; l = (list_T *)alloc_clear(sizeof(list_T) + count * sizeof(listitem_T)); - if (l != NULL) + if (l == NULL) + return NULL; + + list_init(l); + + if (count > 0) { - list_init(l); - - if (count > 0) + listitem_T *li = (listitem_T *)(l + 1); + int i; + + l->lv_len = count; + l->lv_with_items = count; + l->lv_first = li; + l->lv_u.mat.lv_last = li + count - 1; + for (i = 0; i < count; ++i) { - listitem_T *li = (listitem_T *)(l + 1); - int i; - - l->lv_len = count; - l->lv_with_items = count; - l->lv_first = li; - l->lv_u.mat.lv_last = li + count - 1; - for (i = 0; i < count; ++i) - { - if (i == 0) - li->li_prev = NULL; - else - li->li_prev = li - 1; - if (i == count - 1) - li->li_next = NULL; - else - li->li_next = li + 1; - ++li; - } + if (i == 0) + li->li_prev = NULL; + else + li->li_prev = li - 1; + if (i == count - 1) + li->li_next = NULL; + else + li->li_next = li + 1; + ++li; } } + return l; } diff --git a/src/main.c b/src/main.c --- a/src/main.c +++ b/src/main.c @@ -1298,16 +1298,15 @@ main_loop( #endif // Trigger CursorMoved if the cursor moved. - if (!finish_op && ( - has_cursormoved() + if (!finish_op && (has_cursormoved() #ifdef FEAT_PROP_POPUP - || popup_visible + || popup_visible #endif #ifdef FEAT_CONCEAL - || curwin->w_p_cole > 0 + || curwin->w_p_cole > 0 #endif - ) - && !EQUAL_POS(last_cursormoved, curwin->w_cursor)) + ) + && !EQUAL_POS(last_cursormoved, curwin->w_cursor)) { if (has_cursormoved()) apply_autocmds(EVENT_CURSORMOVED, NULL, NULL, @@ -1401,10 +1400,8 @@ main_loop( } #endif - /* - * Before redrawing, make sure w_topline is correct, and w_leftcol - * if lines don't wrap, and w_skipcol if lines wrap. - */ + // Before redrawing, make sure w_topline is correct, and w_leftcol + // if lines don't wrap, and w_skipcol if lines wrap. update_topline(); validate_cursor(); diff --git a/src/ops.c b/src/ops.c --- a/src/ops.c +++ b/src/ops.c @@ -3422,7 +3422,7 @@ free_operatorfunc_option(void) #if defined(FEAT_EVAL) || defined(PROTO) /* - * Mark the global 'operatorfunc' callback with 'copyID' so that it is not + * Mark the global 'operatorfunc' callback with "copyID" so that it is not * garbage collected. */ int diff --git a/src/screen.c b/src/screen.c --- a/src/screen.c +++ b/src/screen.c @@ -1017,7 +1017,7 @@ win_redr_custom( char_u *stl; char_u *p; char_u *opt_name; - int opt_scope = 0; + int opt_scope = 0; stl_hlrec_T *hltab; stl_hlrec_T *tabtab; win_T *ewp; diff --git a/src/terminal.c b/src/terminal.c --- a/src/terminal.c +++ b/src/terminal.c @@ -1234,8 +1234,8 @@ update_cursor(term_T *term, int redraw) gui_mch_flush(); } #endif - // Make sure an invoked autocmd doesn't delete the buffer (and the - // terminal) under our fingers. + // Make sure an invoked autocmd doesn't delete the buffer (and the + // terminal) under our fingers. ++term->tl_buffer->b_locked; // save and restore curwin and curbuf, in case the autocmd changes them diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -696,6 +696,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 934, +/**/ 933, /**/ 932,