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

Add missing error prints to format-specific load/save functions #99758

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Nov 27, 2024

This also updates the documentation.

Note that this doesn't affect load_from_file() because it uses dynamic loaders, while these functions are wrappers around direct calls to functions defined inside modules without using the RresourceLoader API.

This also updates the documentation.

Note that this doesn't affect `load_from_file()` because it uses dynamic loaders,
while these functions are wrappers around direct calls to functions defined
inside modules without using the RresourceLoader API.

Co-authored-by: Hugo Locurcio <[email protected]>
@@ -358,6 +358,7 @@
<param index="0" name="buffer" type="PackedByteArray" />
<description>
Loads an image from the binary contents of a JPEG file.
[b]Note:[/b] This method is only available in engine builds with the JPEG module enabled. By default, the JPEG module is enabled, but it can be disabled at build-time using the [code]module_jpg_enabled=no[/code] SCons option.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[b]Note:[/b] This method is only available in engine builds with the JPEG module enabled. By default, the JPEG module is enabled, but it can be disabled at build-time using the [code]module_jpg_enabled=no[/code] SCons option.
[b]Note:[/b] This method is only available in engine builds with the JPEG module, which is enabled by default. It can be disabled at build-time using the [code]module_jpg_enabled=no[/code] SCons option.

Or similar? For the rest as well...?

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.

3 participants