jadar/DeveloperCapes — pull request #20 by darkhax (2014-01-07 04:32:48+00:00)
Title: Removed basically everything in favor of a better system that is a bit

Hey, saw your code and thought I would make some recommendations. I normally don't code in the same style as the original project by I tried to match the original style to the best of my abilities. This pull request isn't meant to be actually integrated into the project however there are better, more efficient ways of doing this. In the current code there is only one class and three methods that are available. The first is for adding a cape to a specific user. The second is for adding a cape for a group of users. The final method is a utility method to create an array based on a provided url, this method works great with the second as the developer can add capes to users on the fly. As for the resolution of the cape, I don't know what resolutions the current mod supports however this way will support all resolutions as long as they are based on a resolution scaled up to the vanilla counterpart and are within reasonable size limits, basically the same way forge allows for 32, 64, 128, 256 and 512 res textures for items and blocks.

Some of the key differences between the builds, My version does not have support for adding images from locations in the class path or on the hard drive. This can easily be done by adding two more methods that get the image for the ResourceLocation from a file rather than the url. My version also lacks all of the other class files meaning I don't have the updater classes.

As mentioned before this should not be considered an actual pull request but more of a "suggestion" on how to make the project smaller and more efficient.