gunnarmorling/hibernate-validator — comment on commit 801797cc210db25421f4c5a53800d9b68dfe76c7
File:  line 0
By gunnarmorling:

I like the idea of having only interfaces in the public API. It may even be sufficient to only have `ResourceBundleLocator` in the public part, accompanied by a factory for creating instances as needed:

//returns PlatformResourceBundleLocator
ResourceBundleLocator locator = new ResourceBundleLocatorFactory().forBundle("foo");

//returns AggregateResourceBundleLocator
ResourceBundleLocator locator = new ResourceBundleLocatorFactory().forBundles("foo", "bar");

WDYT, should we leave it as is for 4.3 (that is, only move the enumeration into the locator class to get rid of the import) and aim for the interface-only solution in 5.0 in order to avoid breaking existing clients of the concrete classes in 4.3?