Skip to content

Conversation

tony-p
Copy link
Contributor

@tony-p tony-p commented May 16, 2025

Make it impossible to copy the JsonExporter accidentally. The following snippet currently compiles and runs fine, however will not actually add the converters as it is working with a copy, so not produce the desired output.

auto json_exporter = JsonExporter::get();
json_exporter.addConverter<MyType>();

With this change it will throw a compiler error.

@facontidavide facontidavide merged commit 7ffab7e into BehaviorTree:master May 29, 2025
8 of 9 checks passed
@facontidavide
Copy link
Collaborator

thanks!

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