[Proj] Review of proj.4 pull request

support at mnspoint.com support at mnspoint.com
Wed Oct 19 22:55:14 EST 2016


I just hope that this messy untested stuff (PJ_isea.c) is not in the
official distribution of Proj.4 ?! 

Janne. 

----------------------------------- 

Kurt Schwehr kirjoitti 27.09.2016 08:35:

> Try #2 to the proj mailing list now that I've actually confirmed my email address :) 
> 
> There is no rush on this.  It was just a dead code complaint by a linter, not a failure seen with actual code. 
> 
> On Mon, Sep 26, 2016 at 6:16 PM, Nathan Wagner <nw at hydaspes.if.org> wrote:
> 
>> On Mon, Sep 26, 2016 at 06:42:58PM -0500, Howard Butler wrote:
>>>> On Sep 26, 2016, at 4:04 PM, Kurt Schwehr <schwehr at gmail.com> wrote:
>> 
>>>> The code says it was written by Nathan, so that would likely be the
>>>> best one of you to give it a look over, but maybe someone else might
>>>> know.
>> 
>>>> Any chance one of you could take a look at pr 418?  It's my first
>>>> proj.4 patch...  Am I missing anything with this?  Was there a
>>>> missing if?
>> 
>>>> https://github.com/OSGeo/proj.4/pull/418 [1]
>> 
>>> Please bring this up on the proj4 list, as I am happy to approve all
>>> changesets that do not disrupt the tests -- for both good and bad ;)
>> 
>> I'm looking into it.  I'll have an answer later tonight.  Three to four
>> hours.  Preliminary conclusion is that I had no idea what I was thinking
>> when I wrote that code, and the code after the return is the right code,
>> but maybe there's a bug or something that I missed.  My tests don't
>> exercise that code path.
>> 
>> --
>> nw
> 
> -- 
> -- 
> http://schwehr.org 
> _______________________________________________
> Proj mailing list
> Proj at lists.maptools.org
> http://lists.maptools.org/mailman/listinfo/proj

 

Links:
------
[1] https://github.com/OSGeo/proj.4/pull/418
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.maptools.org/pipermail/proj/attachments/20161020/096e281a/attachment.htm 


More information about the Proj mailing list