Herocraft/Heroes — comment on commit 1b2ab0175abe671042f44d8b8470bd998c0e167d
File: src/com/herocraftonline/dev/heroes/persistence/YMLHeroStorage.java line 72
By generalredneck:

It looks like you could use some work still on refactoring this to use other storage systems. For instance, I foresee that you are going to use loadSkillSettings in any system that you create. You may think of refining your interface (abstract class in this case) to micro manage a little. That will also document the code a little better into what you are expecting programmers to fork out when retrieving or sending to storage.

Just thoughts. I hope they help.