On Fri, 29 Dec 2023 at 11:03, Gisle Vanem <gva...@yahoo.no> wrote:

>
> Got it and tried to build it. With clang-cl there is
> this warning:
>    jprint.c(483,40): warning: omitting the parameter name in a function
> definition is a
>    C2x extension   [-Wc2x-extensions]
>      483 | void jstrcache_print_stats(const char *)
>          |
>

Many many thanks! I hope I've fixed that but I had trouble setting up clang
as the compiler ("CC=clang ./configure" didn't do it for me so I admit to
being ignorant about how to set that up).




> With 'cl' it's an error! Simple enough to fix.
> But a 'gnumake -P' on a Makefile with a multi-line macro:
>
> define MK_TEST_C
>    /*
>     * A simple test for loadable .DLLs in GNU-Make.
>     */
>    #include <stdio.h>
>    #include <stdlib.h>
>    #include <string.h>
>    ...
> endef
>
> generates illegal JSON:
>

Thanks for that - Ive made sure to use the string escaper with defines. :-)
You should see the changes pushed to the branch now

Thanks so much for taking a look at it!

Best regards,

Tim Murphy

Reply via email to