[Github-comments] [geany/geany] filetypesprivate.h: Don't include build.h (#2299)

LarsGit223 notifications at xxxxx
Sat Sep 14 10:32:47 UTC 2019


@b4n 

> @LarsGit223 at least projfilecmds is being accessed outside build.c, in project.c, so it's not actually completely private to build.c. If it was, build.h should have had the forward declaration, and build.c the complete one.

Maybe I was not precise enough. The ```GeanyBuildCommand``` pointers are used outside of ```build.c``` but only for:
- address comparison
- calls to g_free
- calls to SETPTR

So the struct items are only used privately in ```build.c```. Therefore a forward declaration seems to be enough, including ```build.h``` is not required (at the moment).

-- 
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/pull/2299#issuecomment-531469012
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190914/6ef782b7/attachment.html>


More information about the Github-comments mailing list