search

orbisgis / cts

Projection library in Java
GNU Lesser General Public License v3.0
50 stars 16 forks source link

System.out.println() statements #66

Closed mainkai closed 7 years ago

mainkai commented 8 years ago

Hey there, thanks for this great project. But could you please remove/comment out the System.out.println() statements in your code? More specifically, I am talking about LambertAzimuthalEqualArea class, inverse() method (line 178).

I assume it was necessary for debugging, but for people using the suite, it can be annyoing.

Regards, Kai

nicolas-f commented 8 years ago

Yes, thank you for this notification. https://github.com/orbisgis/cts/blob/master/src/main/java/org/cts/crs/CompoundCRS.java#L182 should throw an error instead of println also.

mainkai commented 7 years ago

Thanks for adding it to the milestone. When can we expect this to be fixed? I think it is really slowing down my application...

ebocher commented 7 years ago

Fixed

mainkai commented 7 years ago

Thanks for the quick response, but this one seems to remain: https://github.com/orbisgis/cts/blob/master/src/main/java/org/cts/op/projection/LambertAzimuthalEqualArea.java#L171

ebocher commented 7 years ago

done

mainkai commented 7 years ago

Thanks a lot! Will there be a new .jar file that reflects these changes?

nicolas-f commented 7 years ago

A remaining one https://github.com/orbisgis/cts/blob/master/src/main/java/org/cts/CRSHelper.java#L534

Its not a system.log but I don't why this is a warning.