mirror of
https://github.com/bevyengine/bevy
synced 2024-11-22 12:43:34 +00:00
93fc2d12cf
# Objective
The type `AssetLoadError` has `PartialEq` and `Eq` impls, which is
problematic due to the fact that the `AssetLoaderError` and
`AddAsyncError` variants lie in their impls: they will return `true` for
any `Box<dyn Error>` with the same `TypeId`, even if the actual value is
different. This can lead to subtle bugs if a user relies on the equality
comparison to ensure that two values are equal.
The same is true for `DependencyLoadState`,
`RecursiveDependencyLoadState`.
More generally, it is an anti-pattern for large error types involving
dynamic dispatch, such as `AssetLoadError`, to have equality
comparisons. Directly comparing two errors for equality is usually not
desired -- if some logic needs to branch based on the value of an error,
it is usually more correct to check for specific variants and inspect
their fields.
As far as I can tell, the only reason these errors have equality
comparisons is because the `LoadState` enum wraps `AssetLoadError` for
its `Failed` variant. This equality comparison is only used to check for
`== LoadState::Loaded`, which we can easily replace with an `is_loaded`
method.
## Solution
Remove the `{Partial}Eq` impls from `LoadState`, which also allows us to
remove it from the error types.
## Migration Guide
The types `bevy_asset::AssetLoadError` and `bevy_asset::LoadState` no
longer support equality comparisons. If you need to check for an asset's
load state, consider checking for a specific variant using
`LoadState::is_loaded` or the `matches!` macro. Similarly, consider
using the `matches!` macro to check for specific variants of the
`AssetLoadError` type if you need to inspect the value of an asset load
error in your code.
`DependencyLoadState` and `RecursiveDependencyLoadState` are not
released yet, so no migration needed,
---------
Co-authored-by: Joseph <21144246+JoJoJet@users.noreply.github.com>
|
||
|---|---|---|
| .. | ||
| 3d_scene.rs | ||
| 3d_shapes.rs | ||
| 3d_viewport_to_world.rs | ||
| animated_material.rs | ||
| anisotropy.rs | ||
| anti_aliasing.rs | ||
| atmospheric_fog.rs | ||
| auto_exposure.rs | ||
| blend_modes.rs | ||
| bloom_3d.rs | ||
| camera_sub_view.rs | ||
| clearcoat.rs | ||
| color_grading.rs | ||
| deferred_rendering.rs | ||
| depth_of_field.rs | ||
| fog.rs | ||
| fog_volumes.rs | ||
| generate_custom_mesh.rs | ||
| irradiance_volumes.rs | ||
| lighting.rs | ||
| lightmaps.rs | ||
| lines.rs | ||
| load_gltf.rs | ||
| load_gltf_extras.rs | ||
| mesh_ray_cast.rs | ||
| meshlet.rs | ||
| motion_blur.rs | ||
| order_independent_transparency.rs | ||
| orthographic.rs | ||
| parallax_mapping.rs | ||
| parenting.rs | ||
| pbr.rs | ||
| pcss.rs | ||
| post_processing.rs | ||
| query_gltf_primitives.rs | ||
| reflection_probes.rs | ||
| render_to_texture.rs | ||
| rotate_environment_map.rs | ||
| scrolling_fog.rs | ||
| shadow_biases.rs | ||
| shadow_caster_receiver.rs | ||
| skybox.rs | ||
| spherical_area_lights.rs | ||
| split_screen.rs | ||
| spotlight.rs | ||
| ssao.rs | ||
| ssr.rs | ||
| texture.rs | ||
| tonemapping.rs | ||
| transmission.rs | ||
| transparency_3d.rs | ||
| two_passes.rs | ||
| update_gltf_scene.rs | ||
| vertex_colors.rs | ||
| visibility_range.rs | ||
| volumetric_fog.rs | ||
| wireframe.rs | ||