Skip to content

Fix transpilation logic to exclude constant members and add expected output files for classes with constant members#521

Merged
PTKu merged 2 commits intodevfrom
fix-don-t-transpile-constant-members
Apr 1, 2026
Merged

Fix transpilation logic to exclude constant members and add expected output files for classes with constant members#521
PTKu merged 2 commits intodevfrom
fix-don-t-transpile-constant-members

Conversation

@PTKu
Copy link
Copy Markdown
Contributor

@PTKu PTKu commented Apr 1, 2026

This pull request improves the handling of constant members in class transpilation, ensuring that only non-constant fields are included in generated code. It updates both the transpiler logic and the test suite to verify the correct behavior, and adds new sample files to cover this scenario.

Key changes include:

Transpiler logic update:

  • Updated the eligibility check in IsMemberEligibleF to exclude constant fields from transpilation, ensuring that only non-constant members are processed.

Testing improvements:

  • Added a new unit test class_with_constant_members to verify that constant members are correctly excluded from the generated code.
  • Introduced new sample source and expected output files for a class with both constant and non-constant members, validating that only non-constant members appear in the generated Onliners and POCO classes for both ax and tia targets. [1] [2] [3]

These changes ensure that constant members are no longer incorrectly included in the transpiled output, improving the accuracy of the code generation process.

@PTKu PTKu merged commit 8bbd6e4 into dev Apr 1, 2026
2 checks passed
@PTKu PTKu deleted the fix-don-t-transpile-constant-members branch April 1, 2026 12:40
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.

1 participant