search

tzachshabtay / MonoAGS

AGS (Adventure Game Studio) reimagined in Mono
https://tzachshabtay.github.io/MonoAGS/
Artistic License 2.0
28 stars 8 forks source link

Resource loading- refactored to have multiple resource packs #229

Closed tzachshabtay closed 6 years ago

tzachshabtay commented 6 years ago

Refactored the resource loader (based on suggestions from @ivan-mogilko ) to have multiple resource packs (and can be configured to add more). The 2 existing functionalities of loading embedded resources and loading resources from the file system were extracted into 2 built in resource packs.

ghost commented 6 years ago

There's something that attracts my attention here, is that priority is a member of the resource pack interface. May I propose that it might better be a part of the resource pack list, or list item (depends on how they are stored) instead? My reasoning here is that this interface is strictly for reading resources, a resource pack's only concern is about its own resources; and priority is a concept that is applied when several resource packs are used together. Or rather when some external entity (resource manager?) decides an order in which to use these packs.