Skip to content

Remove LoadedAsset::meta #15485

@andriyDev

Description

@andriyDev

What problem does this solve or what need does it fill?

This field is (nearly) unused, and inaccessible. It adds more conceptual overload for no advantage.

What solution would you like?

Delete the field!

Additional context

This may be planned feature work, but I don't think so. Subassets having metadata doesn't make much sense as far as I can tell, and the root asset has its metadata passed in and it isn't changed. The only part that's a little confusing is the populate_hashes argument, but that only affects the loader dependencies, not the asset's meta field.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-AssetsLoad files from disk to use for things like images, models, and soundsC-FeatureA new feature, making something new possibleD-TrivialNice and easy! A great choice to get started with BevyS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!X-ContentiousThere are nontrivial implications that should be thought through

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions