Re: Warning (no error) in regexp.nfa.c included from regexp.c in Tiny build

2023-03-27 Fir de Conversatie Tony Mechelynck
On Sat, Mar 25, 2023 at 12:30 PM Bram Moolenaar  wrote:
>
>
> Tony wrote:
>
> > In file included from regexp.c:2707:
> > regexp_nfa.c: In function ‘nfa_regmatch’:
> > regexp_nfa.c:6696:33: warning: ‘bytelen’ may be used uninitialized
> > [-Wmaybe-uninitialized]
> >  6696 | if (bytelen == 0)
> >   | ^~~~
> > regexp_nfa.c:6679:21: note: ‘bytelen’ declared here
> >  6679 | int bytelen;
> >   | ^~~
> >
> > This was in "make reconfig" at patchlevel 9.0.1425 but maybe appeared 
> > earlier.
>
> I guess you build without syntax highlighting.
>
I don't specifically include or exclude syntax highlighting. IOW,
according to runtime/doc/various.txt:469, my Normal and bigger builds
have it, my smaller builds don't.

This warning did not reappear in the "make reconfig" build I did today
at patchlevel 9.0.1429 (which IIUC means that patch 1427 fixed it) but
I got another one instead (which I described in another email a few
minutes ago, concerning memline.c and possibly os_unix.h) for builds
with +eval only.

Best regards,
Tony.

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAJkCKXvAUjJQSe4x-xNHcTx2V1t%3DLJMi1pPwifAmsBenOYBp7A%40mail.gmail.com.


Compilation warnings in builds with expression evaluation

2023-03-27 Fir de Conversatie Tony Mechelynck
I noticed this in a "make reconfig" at patchlevel 9.0.1429. It may or
may not have happened earlier. Builds without expression evaluation
are not affected.

The following is from a Normal build without the -j argument to make,
to avoid line-interlacing from builds of different modules:

gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK
-I/usr/include/gtk-3.0 -I/usr/include/pango-1.0
-I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include
-I/usr/include/harfbuzz -I/usr/include/freetype2
-I/usr/include/libmount -I/usr/include/blkid -I/usr/include/fribidi
-I/usr/include/cairo -I/usr/include/libpng16 -I/usr/include/pixman-1
-I/usr/include/gdk-pixbuf-2.0 -I/usr/include/gio-unix-2.0
-I/usr/include/wayland -I/usr/include/libxkbcommon
-I/usr/include/atk-1.0 -I/usr/include/at-spi2-atk/2.0
-I/usr/include/at-spi-2.0 -I/usr/include/dbus-1.0
-I/usr/lib64/dbus-1.0/include -pthread-O2 -fno-strength-reduce
-Wall -Wno-deprecated-declarations -D_REENTRANT -U_FORTIFY_SOURCE
-D_FORTIFY_SOURCE=1-o objects/memline.o memline.c
memline.c: In function ‘adjust_text_props_for_delete’:
memline.c:3623:43: warning: ‘this_props_len’ may be used uninitialized
[-Wmaybe-uninitialized]
 3623 | for (done_this = 0; done_this < this_props_len;
  | ~~^~~~
memline.c:3582:17: note: ‘this_props_len’ was declared here
 3582 | int this_props_len;
  | ^~
In file included from vim.h:244,
 from memline.c:45:
memline.c:3630:56: warning: ‘text’ may be used uninitialized
[-Wmaybe-uninitialized]
 3630 | mch_memmove(&prop_this, text + textlen + done_this,
os_unix.h:430:68: note: in definition of macro ‘mch_memmove’
  430 | # define mch_memmove(to, from, len) memmove((char *)(to),
(char *)(from), len)
  |^~~~
memline.c:3583:18: note: ‘text’ was declared here
 3583 | char_u  *text;
  |  ^~~~
memline.c:3630:56: warning: ‘textlen’ may be used uninitialized
[-Wmaybe-uninitialized]
 3630 | mch_memmove(&prop_this, text + textlen + done_this,
  |^
os_unix.h:430:68: note: in definition of macro ‘mch_memmove’
  430 | # define mch_memmove(to, from, len) memmove((char *)(to),
(char *)(from), len)
  |^~~~
memline.c:3584:17: note: ‘textlen’ was declared here
 3584 | size_t  textlen;
  | ^~~

The configuration arguments for this build are set as follows:

export CONF_OPT_GUI='--enable-gui=gtk3'
export CONF_OPT_MULTIBYTE='--enable-multibyte'
export CONF_OPT_AUTOSERVE='--enable-autoservername'
export CONF_OPT_FEAT='--with-features=normal'
export CONF_ARGS2='--with-vim-name=vim-normal'
export CONF_OPT_COMPBY='"--with-compiledby=antoine.mechely...@gmail.com"'


Best regards,
Tony.

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAJkCKXsnCpBJ9zdmPZ4qy0kqODOCOUWxc5ODu17HjL_6aFky8g%40mail.gmail.com.