A side note:
I will be away from tomorrow to monday so I can't check it back before
tuesday.
I have put a note on my agenda to check it again when I'm back.
Cheers,
Daniele
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==
Ing. Daniele Romagnoli
Senior Software Engineer
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
On Thu, Jul 18, 2013 at 6:41 PM, Daniele Romagnoli <
daniele.romagn...@geo-solutions.it> wrote:
> Hi Brett,
> I did a quick look to the patch and it seems good to me.
> The only thing which I didn't understand is why you allow to compare
> DateRange with NumberRange (the latest "else if" returning 1 and the last
> "return -1")
> Ranges to be compared should be of the same type (since an additional
> domain is made of homogeneous objects... numbers or dates or ...)
>
> Can you fix that?
> Cheers,
> Daniele
>
>
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
> ==
> Ing. Daniele Romagnoli
> Senior Software Engineer
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
>
> On Thu, Jul 18, 2013 at 4:29 PM, Brett Walker <brett.wal...@geometryit.com
> > wrote:
>
>> Hi
>>
>> Could someone review PR #237.
>>
>> I found and fixed a build error when using JDK7. It is in unsupported
>> module but it still broke the build.
>>
>> Additional information is found in the commit message of the pull request.
>>
>> Thanks,
>> Brett
>>
>>
>> ------------------------------------------------------------------------------
>> See everything from the browser to the database with AppDynamics
>> Get end-to-end visibility with application monitoring from AppDynamics
>> Isolate bottlenecks and diagnose root cause in seconds.
>> Start your free trial of AppDynamics Pro today!
>>
>> http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
>> _______________________________________________
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>>
>
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
