John Marriott wrote:

> On 30-Nov-2021 08:13, Bram Moolenaar wrote:
> > Patch 8.2.3700
> > Problem:    Text property highlighting continues over breakindent.
> > Solution:   Stop before the end column. (closes #9242)
> > Files:      src/drawline.c, src/testdir/test_textprop.vim,
> >              src/testdir/dumps/Test_prop_linebreak.dump
> >
> >
> After this patch, mingw64 (gcc 11.2.0) throws this warning if 
> FEAT_LINEBREAK is defined but FEAT_PROP_POPUP is not:
> <snip>
> gcc -c -I. -Iproto -DWIN32 -DWINVER=0x0603 -D_WIN32_WINNT=0x0603 
> -DHAVE_PATHDEF -DFEAT_NORMAL -DHAVE_STDINT_H -D__USE_MINGW_ANSI_STDIO 
> -pipe -march=native -Wall -O3 -fomit-frame-pointer -freg-struct-return 
> -fpie -fPIE -DFEAT_GUI_MSWIN -DFEAT_CLIPBOARD drawline.c -o 
> gobjnative/drawline.o
> drawline.c: In function 'win_line':
> drawline.c:269:17: warning: variable 'in_linebreak' set but not used 
> [-Wunused-but-set-variable]
>    269 |     int         in_linebreak = FALSE;   // n_extra set for 
> showing linebreak
>        |                 ^~~~~~~~~~~~
> </snip>
> 
> After enabling FEAT_PROP_POPUP, the linker then throws this error if 
> FEAT_SPELL, FEAT_QUICKFIX or PROTO are not defined (see memline.c, line 
> #3361):

[...]

That probably is the wrong way to add this define.  Not all combinations
are valid.

I'll adjust the #ifdefs, that should fix the first problem.

-- 
Any resemblance between the above views and those of my employer, my terminal,
or the view out my window are purely coincidental.  Any resemblance between
the above and my own views is non-deterministic.  The question of the
existence of views in the absence of anyone to hold them is left as an
exercise for the reader.  The question of the existence of the reader is left
as an exercise for the second god coefficient.  (A discussion of
non-orthogonal, non-integral polytheism is beyond the scope of this article.)
                                                (Ralph Jennings)

 /// 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/20211202113704.915FA1C070D%40moolenaar.net.

Raspunde prin e-mail lui