Patch 9.0.0203
Problem: Confusing variable name.
Solution: Use "prim_aep" instead of "spell_aep".
Files: src/highlight.c
*** ../vim-9.0.0202/src/highlight.c 2022-08-10 20:50:04.233565227 +0100
--- src/highlight.c 2022-08-13 19:07:28.262192052 +0100
***************
*** 2704,2710 ****
hl_combine_attr(int char_attr, int prim_attr)
{
attrentry_T *char_aep = NULL;
! attrentry_T *spell_aep;
attrentry_T new_en;
if (char_attr == 0)
--- 2704,2710 ----
hl_combine_attr(int char_attr, int prim_attr)
{
attrentry_T *char_aep = NULL;
! attrentry_T *prim_aep;
attrentry_T new_en;
if (char_attr == 0)
***************
*** 2732,2753 ****
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! spell_aep = syn_gui_attr2entry(prim_attr);
! if (spell_aep != NULL)
{
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr,
! spell_aep->ae_attr);
! if (spell_aep->ae_u.gui.fg_color != INVALCOLOR)
! new_en.ae_u.gui.fg_color = spell_aep->ae_u.gui.fg_color;
! if (spell_aep->ae_u.gui.bg_color != INVALCOLOR)
! new_en.ae_u.gui.bg_color = spell_aep->ae_u.gui.bg_color;
! if (spell_aep->ae_u.gui.sp_color != INVALCOLOR)
! new_en.ae_u.gui.sp_color = spell_aep->ae_u.gui.sp_color;
! if (spell_aep->ae_u.gui.font != NOFONT)
! new_en.ae_u.gui.font = spell_aep->ae_u.gui.font;
# ifdef FEAT_XFONTSET
! if (spell_aep->ae_u.gui.fontset != NOFONTSET)
! new_en.ae_u.gui.fontset = spell_aep->ae_u.gui.fontset;
# endif
}
}
--- 2732,2753 ----
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! prim_aep = syn_gui_attr2entry(prim_attr);
! if (prim_aep != NULL)
{
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr,
! prim_aep->ae_attr);
! if (prim_aep->ae_u.gui.fg_color != INVALCOLOR)
! new_en.ae_u.gui.fg_color = prim_aep->ae_u.gui.fg_color;
! if (prim_aep->ae_u.gui.bg_color != INVALCOLOR)
! new_en.ae_u.gui.bg_color = prim_aep->ae_u.gui.bg_color;
! if (prim_aep->ae_u.gui.sp_color != INVALCOLOR)
! new_en.ae_u.gui.sp_color = prim_aep->ae_u.gui.sp_color;
! if (prim_aep->ae_u.gui.font != NOFONT)
! new_en.ae_u.gui.font = prim_aep->ae_u.gui.font;
# ifdef FEAT_XFONTSET
! if (prim_aep->ae_u.gui.fontset != NOFONTSET)
! new_en.ae_u.gui.fontset = prim_aep->ae_u.gui.fontset;
# endif
}
}
***************
*** 2777,2813 ****
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! spell_aep = syn_cterm_attr2entry(prim_attr);
! if (spell_aep != NULL)
{
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr,
! spell_aep->ae_attr);
! if (spell_aep->ae_u.cterm.fg_color > 0)
! new_en.ae_u.cterm.fg_color = spell_aep->ae_u.cterm.fg_color;
! if (spell_aep->ae_u.cterm.bg_color > 0)
! new_en.ae_u.cterm.bg_color = spell_aep->ae_u.cterm.bg_color;
! if (spell_aep->ae_u.cterm.ul_color > 0)
! new_en.ae_u.cterm.ul_color = spell_aep->ae_u.cterm.ul_color;
#ifdef FEAT_TERMGUICOLORS
// If both fg and bg are not set fall back to cterm colors.
// Helps for SpellBad which uses undercurl in the GUI.
! if (COLOR_INVALID(spell_aep->ae_u.cterm.fg_rgb)
! && COLOR_INVALID(spell_aep->ae_u.cterm.bg_rgb))
{
! if (spell_aep->ae_u.cterm.fg_color > 0)
new_en.ae_u.cterm.fg_rgb = CTERMCOLOR;
! if (spell_aep->ae_u.cterm.bg_color > 0)
new_en.ae_u.cterm.bg_rgb = CTERMCOLOR;
}
else
{
! if (spell_aep->ae_u.cterm.fg_rgb != INVALCOLOR)
! new_en.ae_u.cterm.fg_rgb = spell_aep->ae_u.cterm.fg_rgb;
! if (spell_aep->ae_u.cterm.bg_rgb != INVALCOLOR)
! new_en.ae_u.cterm.bg_rgb = spell_aep->ae_u.cterm.bg_rgb;
}
! if (spell_aep->ae_u.cterm.ul_rgb != INVALCOLOR)
! new_en.ae_u.cterm.ul_rgb = spell_aep->ae_u.cterm.ul_rgb;
#endif
}
}
--- 2777,2813 ----
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! prim_aep = syn_cterm_attr2entry(prim_attr);
! if (prim_aep != NULL)
{
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr,
! prim_aep->ae_attr);
! if (prim_aep->ae_u.cterm.fg_color > 0)
! new_en.ae_u.cterm.fg_color = prim_aep->ae_u.cterm.fg_color;
! if (prim_aep->ae_u.cterm.bg_color > 0)
! new_en.ae_u.cterm.bg_color = prim_aep->ae_u.cterm.bg_color;
! if (prim_aep->ae_u.cterm.ul_color > 0)
! new_en.ae_u.cterm.ul_color = prim_aep->ae_u.cterm.ul_color;
#ifdef FEAT_TERMGUICOLORS
// If both fg and bg are not set fall back to cterm colors.
// Helps for SpellBad which uses undercurl in the GUI.
! if (COLOR_INVALID(prim_aep->ae_u.cterm.fg_rgb)
! && COLOR_INVALID(prim_aep->ae_u.cterm.bg_rgb))
{
! if (prim_aep->ae_u.cterm.fg_color > 0)
new_en.ae_u.cterm.fg_rgb = CTERMCOLOR;
! if (prim_aep->ae_u.cterm.bg_color > 0)
new_en.ae_u.cterm.bg_rgb = CTERMCOLOR;
}
else
{
! if (prim_aep->ae_u.cterm.fg_rgb != INVALCOLOR)
! new_en.ae_u.cterm.fg_rgb = prim_aep->ae_u.cterm.fg_rgb;
! if (prim_aep->ae_u.cterm.bg_rgb != INVALCOLOR)
! new_en.ae_u.cterm.bg_rgb = prim_aep->ae_u.cterm.bg_rgb;
}
! if (prim_aep->ae_u.cterm.ul_rgb != INVALCOLOR)
! new_en.ae_u.cterm.ul_rgb = prim_aep->ae_u.cterm.ul_rgb;
#endif
}
}
***************
*** 2829,2842 ****
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! spell_aep = syn_term_attr2entry(prim_attr);
! if (spell_aep != NULL)
{
! new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, spell_aep->ae_attr);
! if (spell_aep->ae_u.term.start != NULL)
{
! new_en.ae_u.term.start = spell_aep->ae_u.term.start;
! new_en.ae_u.term.stop = spell_aep->ae_u.term.stop;
}
}
}
--- 2829,2842 ----
new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_attr);
else
{
! prim_aep = syn_term_attr2entry(prim_attr);
! if (prim_aep != NULL)
{
! new_en.ae_attr = ATTR_COMBINE(new_en.ae_attr, prim_aep->ae_attr);
! if (prim_aep->ae_u.term.start != NULL)
{
! new_en.ae_u.term.start = prim_aep->ae_u.term.start;
! new_en.ae_u.term.stop = prim_aep->ae_u.term.stop;
}
}
}
*** ../vim-9.0.0202/src/version.c 2022-08-13 21:34:18.999400776 +0100
--- src/version.c 2022-08-13 21:36:53.243202337 +0100
***************
*** 737,738 ****
--- 737,740 ----
{ /* Add new patch number below this line */
+ /**/
+ 203,
/**/
--
What the word 'politics' means: 'Poli' in Latin meaning 'many' and 'tics'
meaning 'bloodsucking creatures'.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/20220813203754.B81581C091A%40moolenaar.net.