This should remain public, as the main method is not the only way someone might want to run it. Taking a TextureAtlasData is better but the parameter name "outputDir" was more descriptive. If outputDir can be null, the javadoc should explain that. Otherwise I'd prefer not to have empty parameter javadocs.