Skip to content

Updated to support C api include links#198

Merged
JCash merged 1 commit intomasterfrom
c-language-fixes
Feb 17, 2026
Merged

Updated to support C api include links#198
JCash merged 1 commit intomasterfrom
c-language-fixes

Conversation

@JCash
Copy link
Contributor

@JCash JCash commented Feb 17, 2026

No description provided.

<th></th>
<th></th>
</tr>
{%- if ref.info.language == "C++" and ref.info.namespace != "" -%}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only show namespace if it's an actual namespace.

The "namespace" field has been overtaken by other functions as well, so we can't simply check if it's not empty.

</tr>
{%- endunless -%}
{%- endif -%}
{%- if ref.info.include and ref.info.include != "" -%}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The include field wasn't set, thus the link was shown, but with no text.

To make it clearer, only show an include, if it actually has one.


# set api type
api["info"]["type"] = "Defold " + api["info"]["language"]
language = api["info"]["language"]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tiny cleanup to avoid getting it again later on.

# generate include path for C++ files
if language == "C++":
# generate include path for C/C++ files
if language in ("C++", "C"):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix for the include file link for C files.

@JCash JCash requested a review from britzl February 17, 2026 13:35
@JCash JCash merged commit 0dc6f31 into master Feb 17, 2026
1 check passed
@JCash JCash deleted the c-language-fixes branch February 17, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants