Add `System::dispose`

[Thomas Schaller]

Requested by @Frizi.

System::dispose would be called by Dispatcher::dispose and can be used to clean up externally allocated resources.

Unresolved

• Signature: fn dispose(&self, world: &World) / fn dispose(&self, world: &mut World) / fn dispose(&mut self, world: &World) / fn dispose(&mut self, world: &mut World)
• Is it mandatory to call it? How to make sure it gets called?

[Lucio Franco]

@torkleyy curious what is the difference between this and drop?

[Thomas Schaller]

It has access to World, which is required for access to rendy::Factory in this case.

[Lucio Franco]

@torkleyy that makes sense, I would imagine in that case a fn dispose(&mut self, world: &mut World) might make sense here.

[Thomas Schaller]

Yeah, I think that makes sense.

[Thomas Schaller]

Now, how do we make sure it gets called?

One option would be to set a flag when it gets called, and check it is set in Dispatcher::drop.