-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
AssetServer lifetimes can probably be simplified #15541
Copy link
Copy link
Closed
Labels
A-AssetsLoad files from disk to use for things like images, models, and soundsLoad files from disk to use for things like images, models, and soundsD-Domain-AgnosticCan be tackled by anyone with generic programming or Rust skillsCan be tackled by anyone with generic programming or Rust skillsD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-UncontroversialThis work is generally agreed uponThis work is generally agreed upon
Description
I might be missing something, but I think these can be simplified. There's also other functions in the same file that has unused
'alifetimes.
Originally posted by @kristoff3r in #15533 (review)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-AssetsLoad files from disk to use for things like images, models, and soundsLoad files from disk to use for things like images, models, and soundsD-Domain-AgnosticCan be tackled by anyone with generic programming or Rust skillsCan be tackled by anyone with generic programming or Rust skillsD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-UncontroversialThis work is generally agreed uponThis work is generally agreed upon