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

Core: Add Callable.create static method for Variant callables #88948

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

dalexeev
Copy link
Member

@dalexeev dalexeev commented Feb 28, 2024

@dalexeev dalexeev added this to the 4.3 milestone Feb 28, 2024
@dalexeev dalexeev requested a review from a team February 28, 2024 09:57
@dalexeev dalexeev requested review from a team as code owners February 28, 2024 09:57
core/variant/callable.cpp Outdated Show resolved Hide resolved
core/variant/callable.cpp Show resolved Hide resolved
@dalexeev dalexeev force-pushed the core-add-callable-create branch from ec5527f to 0cb7de0 Compare February 28, 2024 13:25
Copy link
Member

@adamscott adamscott left a comment

Choose a reason for hiding this comment

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

Great way to fix #87122!

doc/classes/Callable.xml Outdated Show resolved Hide resolved
@@ -160,7 +188,11 @@
<method name="is_custom" qualifiers="const">
<return type="bool" />
<description>
Returns [code]true[/code] if this [Callable] is a custom callable. Custom callables are created from [method bind] or [method unbind]. In GDScript, lambda functions are also custom callables.
Returns [code]true[/code] if this [Callable] is a custom callable. Custom callables are used:
Copy link
Contributor

Choose a reason for hiding this comment

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

I may be better to reword the list to describe how custom Callables are created, how to create and obtain them, rather than list what they're used for.

I have no suggestions at the moment.

doc/classes/Callable.xml Outdated Show resolved Hide resolved
doc/classes/Callable.xml Outdated Show resolved Hide resolved
doc/classes/Callable.xml Outdated Show resolved Hide resolved
doc/classes/Callable.xml Outdated Show resolved Hide resolved
@dalexeev dalexeev force-pushed the core-add-callable-create branch from 0cb7de0 to d90c9db Compare February 29, 2024 07:45
@akien-mga akien-mga merged commit 9278856 into godotengine:master Feb 29, 2024
16 checks passed
@dalexeev dalexeev deleted the core-add-callable-create branch February 29, 2024 12:58
@akien-mga
Copy link
Member

Thanks!

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.

No warning when getting built in Callable from Dictionary
6 participants