Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Blueprint Nativize #55

Merged
merged 2 commits into from
Sep 29, 2021
Merged

Support Blueprint Nativize #55

merged 2 commits into from
Sep 29, 2021

Conversation

brwarner
Copy link
Contributor

@brwarner brwarner commented Sep 28, 2021

Previously, the plugin generated non-compilable C++ code when using Blueprint Nativize. There were two culprits.

  1. A method on IArticyReflectable which shared its name GetClass with a method on UObject, creating an ambiguous call.
  2. A number of BlueprintCallable methods which were protected, not public, creating compile errors (it seems protected variables are fine though, as nativize builds a workaround).

There may be more issues, but I haven't identified any more in testing.

@brwarner brwarner added the 1.3.1 label Sep 28, 2021
@brwarner brwarner self-assigned this Sep 28, 2021
@brwarner brwarner changed the base branch from master to development September 28, 2021 20:05
@brwarner brwarner merged commit c3981e8 into development Sep 29, 2021
@brwarner brwarner deleted the blueprint-nativize-fix branch September 29, 2021 14:27
@brwarner brwarner added this to the v1.3.1 milestone Oct 8, 2021
@brwarner brwarner removed the 1.3.1 label Oct 8, 2021
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