>
> Briefly GeoTools.init() was going to register an ImagingListener. The
>> rejected pull request added GeoTools.init() to each test case @Before
>> method.
>>
>
> Mind, I did not mean to reject the pull request, just wanted to express
> concern on its (lack of) future maintability
>
Based on your feedback I am happy to reject the pull request. I actually
had both ideas yesterday and took the least short-term risk in order to try
out the change and confirm it does not have undue side effects.
In todays meeting we hit on another approach. In the static init of our
>> logging class I will check if JAI has an ImagingListener, and if it is null
>> I will add in our own.
>>
>
> Right. Anyone knows if that might present issues we haven't considered?
>
I think it will be fine, I was more concerned that introducing
GeoServer.int() everywhere would slow down tests - as it plays some silly
games to try and connect to first commons logging and then log4j before
defaulting to Java logging.
--
Jody
------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
