<div dir="ltr">Jeff,<div><br><div>Given that the proposal does not address the issue of backwards compatibility with the current php mapscript, I suggest that (1) the whole php+swig code reside in its own mapscript/(phpng|php7) directory instead of having #ifdefs in the current php mapscript code, and that (2) the module be renamed differently than the current one. When/if a backwards compatibility layer is worked on, that compatibility layer could take on the current name.</div><div><br></div><div>regards,</div><div><br></div><div>--</div><div>thomas</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 3, 2017 at 4:53 PM Jeff McKenna <<a href="mailto:jmckenna@gatewaygeomatics.com">jmckenna@gatewaygeomatics.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Update: SWIG-3.0.11 was released recently on 29 December containing the<br class="gmail_msg">
full PHP 7 support.<br class="gmail_msg">
<br class="gmail_msg">
-jeff<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
On 2016-12-20 5:58 PM, Jeff McKenna wrote:<br class="gmail_msg">
> Regarding the clone() keyword, I looked closer at the MapServer source,<br class="gmail_msg">
> and in fact Java MapScript faced the same issue apparently, and handled<br class="gmail_msg">
> that by defining a new function name for those use cases (cloning map,<br class="gmail_msg">
> layer, class, style, shape instances) such as:<br class="gmail_msg">
><br class="gmail_msg">
> #ifdef SWIGJAVA<br class="gmail_msg">
>   %newobject cloneMap;<br class="gmail_msg">
>   mapObj *cloneMap()<br class="gmail_msg">
> #else<br class="gmail_msg">
>   %newobject clone;<br class="gmail_msg">
>   mapObj *clone()<br class="gmail_msg">
> #endif<br class="gmail_msg">
><br class="gmail_msg">
> So we would do the same for SWIGPHP (reference them in the /swiginc/<br class="gmail_msg">
> files and define those functions in /mapscript/php/phpextend.i<br class="gmail_msg">
><br class="gmail_msg">
> (I've updated the RFC with those notes)<br class="gmail_msg">
><br class="gmail_msg">
> Thanks for the feedback Ducky!<br class="gmail_msg">
><br class="gmail_msg">
> -jeff<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> On 2016-12-19 10:49 AM, Dan Little wrote:<br class="gmail_msg">
>> I'm generally positive about this in the hopes that it means the<br class="gmail_msg">
>> PHP/MapScript+SWIG will ensure PHP gets feature parity with the other<br class="gmail_msg">
>> versions of MapScript.  I also am a fan of what looks to be a more<br class="gmail_msg">
>> consistent syntax to modern PHP.<br class="gmail_msg">
>><br class="gmail_msg">
>> I'll venture an opinion/suggestions on the reserved word cases<br class="gmail_msg">
>> (specifically clone and close):   Is there a way to alias those<br class="gmail_msg">
>> function names to something other than "c_..." ?  If so, I've seen<br class="gmail_msg">
>> some convention such as "clone_" or "close_", and more PHP-y might be<br class="gmail_msg">
>> something to the tune of "cloneInstance()" or "closeXYZ" where XYZ is<br class="gmail_msg">
>> actual context appropriate (closeStream, closeConnection, etc.).<br class="gmail_msg">
>><br class="gmail_msg">
>> Cheers.<br class="gmail_msg">
>><br class="gmail_msg">
>> On Mon, Dec 19, 2016 at 9:47 AM, Dan Little <<a href="mailto:theduckylittle@gmail.com" class="gmail_msg" target="_blank">theduckylittle@gmail.com</a>><br class="gmail_msg">
>> wrote:<br class="gmail_msg">
>>> I'm generally positive about this in the hopes that it means the<br class="gmail_msg">
>>> PHP/MapScript+SWIG will ensure PHP gets feature parity with the other<br class="gmail_msg">
>>> versions of MapScript.  I also am a fan of what looks to be a more<br class="gmail_msg">
>>> consistent syntax to modern PHP.<br class="gmail_msg">
>>><br class="gmail_msg">
>>> I'll venture an opinion/suggestions on the reserved word cases<br class="gmail_msg">
>>> (specifically clone and close):   Is there a way to alias those<br class="gmail_msg">
>>> function names to something other than "c_..." ?  If so, I've seen<br class="gmail_msg">
>>> some convention such as "clone_" or "close_", and more PHP-y might be<br class="gmail_msg">
>>> something to the tune of "cloneInstance()" or "closeXYZ" where XYZ is<br class="gmail_msg">
>>> actual context appropriate (closeStream, closeConnection, etc.).<br class="gmail_msg">
>>><br class="gmail_msg">
>>> Cheers.<br class="gmail_msg">
>>><br class="gmail_msg">
>>> On Fri, Dec 16, 2016 at 4:32 PM, Jeff McKenna<br class="gmail_msg">
>>> <<a href="mailto:jmckenna@gatewaygeomatics.com" class="gmail_msg" target="_blank">jmckenna@gatewaygeomatics.com</a>> wrote:<br class="gmail_msg">
>>>> Hi all,<br class="gmail_msg">
>>>><br class="gmail_msg">
>>>> I've created an RFC draft for the PHP 7 SWIG potential changes:<br class="gmail_msg">
>>>> <a href="http://mapserver.org/development/rfc/ms-rfc-117.html" rel="noreferrer" class="gmail_msg" target="_blank">http://mapserver.org/development/rfc/ms-rfc-117.html</a><br class="gmail_msg">
>>>><br class="gmail_msg">
>>>> Please pay attention to the "Limitations" section, as there are some<br class="gmail_msg">
>>>> items<br class="gmail_msg">
>>>> that need discussion.  I will plan to discuss this for sure at the<br class="gmail_msg">
>>>> next PSC<br class="gmail_msg">
>>>> meeting on 19 January, so we have lots of time to discuss,<br class="gmail_msg">
>>>> especially as we<br class="gmail_msg">
>>>> are entering the holidays.<br class="gmail_msg">
>>>><br class="gmail_msg">
>>>> Wishing the whole MapServer community a truly happy and merry<br class="gmail_msg">
>>>> Christmas!  :)<br class="gmail_msg">
>>>><br class="gmail_msg">
>>>> -jeff<br class="gmail_msg">
>>>><br class="gmail_msg">
>>>><br class="gmail_msg">
><br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
mapserver-dev mailing list<br class="gmail_msg">
<a href="mailto:mapserver-dev@lists.osgeo.org" class="gmail_msg" target="_blank">mapserver-dev@lists.osgeo.org</a><br class="gmail_msg">
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a></blockquote></div>