jakubsynowiec/tomighty — comment on commit 56c8affcce3099f141b058d951da0d6a4c5fc3c0
File: src/main/java/org/tomighty/i18n/Messages.java line 39
By ccidral:

`getDefaultLanguageCountry()` is only called by `loadLanguage()` and `getDefaultResourceName`, and `getDefaultResourceName` is only called by `loadLanguage()`, so you can pass the locale around by parameter which is better than keep the locale in a mutable field here. After all the current locale is determined by the `Options` object. This makes more sense because the current locale is an option setting.

My point is, by removing the `locale` field we would have one less mutable state to care about in this class. Just my 2 cents.