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

16 views
Skip to first unread message

Tony Mechelynck

unread,
Mar 23, 2023, 12:48:07 PM3/23/23
to vim_dev
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.

Best regards,
Tony.

Bram Moolenaar

unread,
Mar 25, 2023, 7:30:19 AM3/25/23
to vim...@googlegroups.com, Tony Mechelynck
I guess you build without syntax highlighting.

--
A real patriot is the fellow who gets a parking ticket and rejoices
that the system works.


/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///

Tony Mechelynck

unread,
Mar 27, 2023, 8:14:01 PM3/27/23
to Bram Moolenaar, vim...@googlegroups.com
On Sat, Mar 25, 2023 at 12:30 PM Bram Moolenaar <Br...@moolenaar.net> 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.
Reply all
Reply to author
Forward
0 new messages