[Github-comments] [geany/geany-plugins] cppcheck: Set Geany include path for it to find header files (#947)

Colomban Wendling notifications at xxxxx
Fri Feb 7 13:39:49 UTC 2020


b4n commented on this pull request.



> @@ -0,0 +1,18 @@
+dnl taken from Autoconf's m4sh.m4, GPLv3+
+m4_ifndef([AS_VAR_COPY], [
+m4_define([AS_VAR_COPY],
+[AS_LITERAL_WORD_IF([$1[]$2], [$1=$$2], [eval $1=\$$2])])
+])

I'm not totally sure about this part: it might not be needed, and if it is I'm not sure `AS_LITERAL_WORD_IF` would be available.  This code is what Autoconf has nowadays, but the inspiration I got from cryptsetup has `AS_LITERAL_IF` instead (but it seems to be buggy in some cases if I trust the documentation).
Anyway, we can fix this if a problem arises

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/947#pullrequestreview-355160747
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20200207/b31ee794/attachment.htm>


More information about the Github-comments mailing list