The current API reads: public interface ParameterizedBridge { //TODO inject Properties? since the annotations cannot support Object attribute? void setParameterValues(Map parameters); }
Shall we apply the TODO, or at least change it to specify the expected Map types? public interface ParameterizedBridge { void setParameterValues(Map<String,String> parameters); } Using generics we would keep backwards compatibility. I'm looking at this because we make type mistakes ourselves; for example the example PaddedIntegerBridge is broken: public void setParameterValues(Map parameters) { Object padding = parameters.get( PADDING_PROPERTY ); if (padding != null) this.padding = (Integer) padding; <<<<<<< Cast exception! } Cheers, Sanne _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev