The change makes sense to me. I wonder, do we feel the interface is widely
used enough to try to make the transition a bit smoother? Something like
adding a new interface, deprecating the first and supporting both for now?
Just a question.
On Fri, Nov 29, 2013 at 7:37 AM, Andrea Aime
<[email protected]>wrote:
> Hi,
> StreamingRenderer allows listeners to be attached to monitor the rendering
> progress, currently they have the following interface:
>
> public interface RenderListener {
> public void featureRenderer( SimpleFeature feature );
> public void errorOccurred( Exception e);
> }
>
> Since it talks simple features only, the StreamingRenderer just does not
> perform the notifications
> if the feature being rendered is not a simple one.
>
> Do you mind if, on trunk, we make it a bit more general, and allow generic
> Feature instead:
>
> public interface RenderListener {
> public void featureRenderer( Feature feature );
> public void errorOccurred( Exception e);
> }
>
> It's an API change (small one). Do you believe a GSIP is needed?
>
> Cheers
> Andrea
>
> --
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
> ==
>
> Ing. Andrea Aime
> @geowolf
> Technical Lead
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
> mob: +39 339 8844549
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
>
> ------------------------------------------------------------------------------
> Rapidly troubleshoot problems before they affect your business. Most IT
> organizations don't have a clear picture of how application performance
> affects their revenue. With AppDynamics, you get 100% visibility into your
> Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics
> Pro!
> http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk
> _______________________________________________
> GeoTools-Devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
--
*Justin Deoliveira*
Vice President, Engineering | Boundless
[email protected]
@j_deolive <https://twitter.com/j_deolive>
------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT
organizations don't have a clear picture of how application performance
affects their revenue. With AppDynamics, you get 100% visibility into your
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
