Skip to content

Fix UPROPERTY initialization and new Blueprint Library for FContextBase#3287

Merged
JasonAtClockwork merged 5 commits into
masterfrom
jlarabie/codegen-blueprint
Oct 1, 2025
Merged

Fix UPROPERTY initialization and new Blueprint Library for FContextBase#3287
JasonAtClockwork merged 5 commits into
masterfrom
jlarabie/codegen-blueprint

Conversation

@JasonAtClockwork

@JasonAtClockwork JasonAtClockwork commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

Description of Changes

Closes: https://github.com/clockworklabs/SpacetimeDBPrivate/issues/2058

  • Updated the generation code to setup basic initialization to properties for Unreal Blueprints
  • Added new Blueprint library to interact with FContextBase to allow access to inherited properties from all contexts in Blueprint

API and ABI breaking changes

No breaking changes

Expected complexity level and risk

2 - Updates the generation code

Testing

  • Ran Unreal tests to confirm no breaking changes

@jdetter jdetter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Few questions on this one, no objections so far 👍

Comment thread crates/codegen/src/unrealcpp.rs Outdated
Comment thread crates/codegen/src/unrealcpp.rs

@jdetter jdetter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I haven't tested the changes myself but this looks like a good change to me. Thanks @JasonAtClockwork! Can you make sure to check the duplicate semicolon thing one more time before merging? Thanks 👍

Comment thread crates/codegen/src/unrealcpp.rs Outdated
Comment thread crates/codegen/src/unrealcpp.rs
@JasonAtClockwork

Copy link
Copy Markdown
Contributor Author

I haven't tested the changes myself but this looks like a good change to me. Thanks @JasonAtClockwork! Can you make sure to check the duplicate semicolon thing one more time before merging? Thanks 👍

Small repairs pushed and the float/double were functional but not as clean as they could be

@JasonAtClockwork JasonAtClockwork added this pull request to the merge queue Oct 1, 2025
Merged via the queue into master with commit f34ffd0 Oct 1, 2025
25 of 26 checks passed
@bfops bfops deleted the jlarabie/codegen-blueprint branch May 6, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants