<div dir="ltr"><div>Hi Seth: I merged mapserver/main into my config branch and updated the msLoadMap() signature to take a configObj *. It just creates a reference to an immutable configObj that can be referenced as needed. I also updated swiginc/map.i to take an optional second parameter. I haven't done anything with msLoadMapFromString() as yet. I'll have to look at why tests are failing but IMHO this should be stable enough to start doing some testing if you wanted.</div><div><br></div><div>--Steve</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 1, 2021 at 6:10 PM Steve Lime <<a href="mailto:sdlime@gmail.com">sdlime@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Yes, it’s stable although I haven’t changed function signatures for msLoadMap() in the C code so that probably should happen first.</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 1, 2021 at 2:07 AM Seth G <<a href="mailto:sethg@geographika.co.uk" target="_blank">sethg@geographika.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u><div><div>HI Steve,<br></div><div><br></div><div>I think that should be fine for all the MapScripts.<br></div><div>The C# ones have a second constructor [1], but this could ignore the configObj, or have it as a third parameter. <br></div><div><br></div><div>Is <a href="https://github.com/MapServer/MapServer/pull/6303" target="_blank">https://github.com/MapServer/MapServer/pull/6303</a> stable enough to create a SWIG configObj wrapper for testing?<br></div><div><br></div><div>Seth<br></div><div><br></div><div><br></div><div>[1] <a href="https://github.com/MapServer/MapServer/blob/e48c5cb1fadec9b3f78b256e121093715b428c1a/mapscript/swiginc/map.i#L44" target="_blank">https://github.com/MapServer/MapServer/blob/e48c5cb1fadec9b3f78b256e121093715b428c1a/mapscript/swiginc/map.i#L44</a></div></div><div><div><br></div><div id="gmail-m_7018477676155993365m_4584141778676444965sig62266145"><div>--<br></div><div>web:<a href="http://geographika.co.uk" target="_blank">http://geographika.co.uk</a><br></div><div>twitter: @geographika<br></div></div><div><br></div><div><br></div><div>On Wed, Jun 30, 2021, at 7:54 PM, Steve Lime wrote:<br></div><blockquote type="cite" id="gmail-m_7018477676155993365m_4584141778676444965qt"><div dir="ltr"><div>Seth: Do you think this could be workable?<br></div><div><br></div><div><div>  $config = new mapscript::configObj('path to config file');<br></div><div>  $map = new mapscript::mapObj('my.map', $config);<br></div></div><div><br></div><div>Seems like we could update the MapScript mapObj constructors to take an optional configObj parameter with default of NULL. Destroying a mapObj doesn't destroy the config (it has a longer scope).<br></div><div><br></div><div>--Steve </div></div></blockquote></div></blockquote></div></div>
</blockquote></div></div>