search

shawn42 / gamebox

A game template for building and distributing Gosu apps.
267 stars 29 forks source link

Refactored ConfigManager #73

Closed burtlo closed 12 years ago

burtlo commented 12 years ago

As ConfigManager did not use initialization other then to load the data I changed the class to use lazy-loading for the configuration.

shawn42 commented 12 years ago

Looks good. Nice to see someone cleaning up some of the cruft that has been built up over the years.

shawn42 commented 12 years ago

Maybe we could look at using something like the configural gem for determining where to store these files? WDYT?

burtlo commented 12 years ago

At the moment, I have no idea what configuration the config manager holds. This was an easy location to start some clean up and learning.

As far as the Configural gem, it seems like it would be a good fit. Again, I'll have a better opinion after I look around more.