mirror of
https://github.com/bevyengine/bevy.git
synced 2026-07-03 17:24:00 -04:00
andriyDev
d196050db1
# Objective - Previously, asset processors needed to be **fully specified**. This meant writing met files by hand was incredibly cumbersome. - In addition, using `core::any::type_name` is generally undesirable since it is only intended for debugging. ## Solution - Require `TypePath` impl on `AssetLoader`, `AssetTransformer`, `AssetSaver`, and `AssetProcessor`. - Register loaders and processors by their `TypePath::type_path`. This makes the path format stable. - Additionally register the processors by their `TypePath::short_type_path`, and ensure there are no other processors with the same `short_type_path`. If a user tries to use an ambiguous short type path, we list out all full type paths. Note: I left doing this with asset loaders as a future PR. There's more complexity there (since we have to deal with pre-registering asset loaders), and this seems like the bigger "bang for our buck". ## Testing - I ran the asset_processing example. No change. - I changed the asset processing example to use the short type path. It still behaved as expected. - I created a new AssetTransformer in `asset_processing::sneaky::CoolTextTransformer` and a new processor that looked identical to the regular one (but using the sneaky transformer). It printed an error, listing out the full paths of both processors! --------- Co-authored-by: Carter Anderson <mcanders1@gmail.com>
13 lines
335 B
Plaintext
13 lines
335 B
Plaintext
(
|
|
meta_format_version: "1.0",
|
|
asset: Process(
|
|
processor: "LoadTransformAndSave<CoolTextLoader, CoolTextTransformer, CoolTextSaver>",
|
|
settings: (
|
|
loader_settings: (),
|
|
transformer_settings: (
|
|
appended: "X",
|
|
),
|
|
saver_settings: (),
|
|
),
|
|
),
|
|
) |