Hi Aaron, I've had another look at this. In general, I prefer this to remain a Map. Some users use jOOQ with Spring and they might be better off if Configuration.data is a Map. Java has lived with unsafe maps (or Properties) for a long time. I don't see an immediate need for a change in this case.
Cheers Lukas 2012/6/8 digulla <[email protected]>: > Hi Lukas, > > Please consider using a TypedMap for Configuration.getData(). See > here: http://blog.pdark.de/2010/05/28/type-safe-object-map/ > > The API isn't perfect at this time; the problem is the put(String, Object) > method which can be called accidentally. > > There are two ways to improve this: > > 1. Don't implement Map<> > > 2. Throw an exception in put(String, Object) to make sure only the type safe > method is ever called. > > Regards, > > A. Digulla
