<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=793526" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/b4n">@b4n</a></p>
<blockquote>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=9009011" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/LarsGit223">@LarsGit223</a> 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.</p>
</blockquote>
<p>Maybe I was not precise enough. The <code>GeanyBuildCommand</code> pointers are used outside of <code>build.c</code> but only for:</p>
<ul>
<li>address comparison</li>
<li>calls to g_free</li>
<li>calls to SETPTR</li>
</ul>
<p>So the struct items are only used privately in <code>build.c</code>. Therefore a forward declaration seems to be enough, including <code>build.h</code> is not required (at the moment).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2299?email_source=notifications&email_token=AAIOWJYXJAOBRYNSORTQGWDQJS4U5A5CNFSM4IWNUYZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6WZFVA#issuecomment-531469012">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ5Z35UHDXT4BWYCPMTQJS4U5ANCNFSM4IWNUYZA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ2MZLLA4EQW22IZFQTQJS4U5A5CNFSM4IWNUYZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6WZFVA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2299?email_source=notifications\u0026email_token=AAIOWJYXJAOBRYNSORTQGWDQJS4U5A5CNFSM4IWNUYZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6WZFVA#issuecomment-531469012",
"url": "https://github.com/geany/geany/pull/2299?email_source=notifications\u0026email_token=AAIOWJYXJAOBRYNSORTQGWDQJS4U5A5CNFSM4IWNUYZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6WZFVA#issuecomment-531469012",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>