Reminder: when changing an API, please remember to do the "deprecate now, delete
later" procedure unless the change conflict with that.
In the recent build failure case, GeoAPI Telephone interface was changed from:
String getVoice();
to:
Collection<String> getVoice();
It should have been:
/**
* @deprecated Replaced by [EMAIL PROTECTED] #getVoices}.
*/
String getVoice();
/**
* @since GeoAPI 2.1
*/
Collection<String> getVoices();
Please remember that when there is such an API change, the compiler will not
catch every broken dependencies. With the original change (Collection<String>
getVoice(), singular form) we got test failure in WMS1_1_1_OnlineTest. This
failure was unoticed by the compiler because 'getVoice()' return value was
assigned to an Object, and unoticed by Maven because this is a online test.
Lets avoid brutal transition when we can. Deprecate first, delete later.
Martin
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
