On 20/12/2023 23:13, Luc Van Oostenryck wrote: [snip] > The following patch is a bit ugly but should solve the problem here above. > It's hasn't had much testing, only the testsuite. > > Cheers, > -- Luc > > >>From 2d1feff1551a97fdfd1149f4772331c540e638c9 Mon Sep 17 00:00:00 2001 > From: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> > Date: Wed, 20 Dec 2023 22:00:05 +0100 > Subject: [PATCH] handle protoype scope > Hi Luc, This looks good and _much_ better than my initial attempt! ;) I tried your patch and everything looks good on the sparse repo. On Git, however, things didn't go quite so well, thus: $ make V=1 sparse >ssp-out 2>&1 $ tail -4 ssp-out >add-patch.sp add-patch.c: note: in included file (through git-compat-util.h): /usr/include/regex.h:682:41: error: Variable length array is used. make: *** [Makefile:3234: add-patch.sp] Error 1 $ I think this is caused by -Wvla, which git passes to gcc, along with sparse (perhaps) not supporting a GCC #pragma; just after the _REGEX_NELTS definition in the header file we find: #if defined __GNUC__ && 4 < __GNUC__ + (6 <= __GNUC_MINOR__) # pragma GCC diagnostic push # pragma GCC diagnostic ignored "-Wvla" #endif But I haven't had time to think about this too deeply. It's nearly 2am and I need to get some sleep - I just wanted to report this ASAP. Thanks for the patch. ATB, Ramsay Jones
