Skip to content

Logic in reload_internal is confusing and needs to be commented #21222

@alice-i-cecile

Description

@alice-i-cecile

Not related to this PR. Dang this needs a comment. It took me a few minutes to work out why this is here. Quite confusing!

The reason: the first section is trying to reload all the different handles to a path if there are multiple asset types. The second section is for if the root asset has been dropped, but any labeled assets for that path are still alive - we just try to load the path without any handle type hoping we get all its labeled assets. This breaks down if those labeled assets come from different root asset types!

Originally posted by @andriyDev in #21218 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-AssetsLoad files from disk to use for things like images, models, and soundsD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions