{"action":"created","issue":{"url":"https://api.github.com/repos/jaeandersson/swig/issues/40","labels_url":"https://api.github.com/repos/jaeandersson/swig/issues/40/labels{/name}","comments_url":"https://api.github.com/repos/jaeandersson/swig/issues/40/comments","events_url":"https://api.github.com/repos/jaeandersson/swig/issues/40/events","html_url":"https://github.com/jaeandersson/swig/pull/40","id":103573475,"number":40,"title":"Module support","user":{"login":"dcb210","id":825550,"avatar_url":"https://avatars.githubusercontent.com/u/825550?v=3","gravatar_id":"","url":"https://api.github.com/users/dcb210","html_url":"https://github.com/dcb210","followers_url":"https://api.github.com/users/dcb210/followers","following_url":"https://api.github.com/users/dcb210/following{/other_user}","gists_url":"https://api.github.com/users/dcb210/gists{/gist_id}","starred_url":"https://api.github.com/users/dcb210/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcb210/subscriptions","organizations_url":"https://api.github.com/users/dcb210/orgs","repos_url":"https://api.github.com/users/dcb210/repos","events_url":"https://api.github.com/users/dcb210/events{/privacy}","received_events_url":"https://api.github.com/users/dcb210/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-08-27T19:00:44Z","updated_at":"2015-08-31T17:26:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jaeandersson/swig/pulls/40","html_url":"https://github.com/jaeandersson/swig/pull/40","diff_url":"https://github.com/jaeandersson/swig/pull/40.diff","patch_url":"https://github.com/jaeandersson/swig/pull/40.patch"},"body":"These commits implement module support for Matlab.  Note that I had to introduce a new member variable, swigPtr, to handle cross module pointers.  Also, the mex files must remain locked in order to support the cross-module type table.  A better implementation may exist, but the submitted changes allow me to run our standard test script in Matlab without runtime errors. "},"comment":{"url":"https://api.github.com/repos/jaeandersson/swig/issues/comments/136435083","html_url":"https://github.com/jaeandersson/swig/pull/40#issuecomment-136435083","issue_url":"https://api.github.com/repos/jaeandersson/swig/issues/40","id":136435083,"user":{"login":"dcb210","id":825550,"avatar_url":"https://avatars.githubusercontent.com/u/825550?v=3","gravatar_id":"","url":"https://api.github.com/users/dcb210","html_url":"https://github.com/dcb210","followers_url":"https://api.github.com/users/dcb210/followers","following_url":"https://api.github.com/users/dcb210/following{/other_user}","gists_url":"https://api.github.com/users/dcb210/gists{/gist_id}","starred_url":"https://api.github.com/users/dcb210/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcb210/subscriptions","organizations_url":"https://api.github.com/users/dcb210/orgs","repos_url":"https://api.github.com/users/dcb210/repos","events_url":"https://api.github.com/users/dcb210/events{/privacy}","received_events_url":"https://api.github.com/users/dcb210/received_events","type":"User","site_admin":false},"created_at":"2015-08-31T17:26:33Z","updated_at":"2015-08-31T17:26:33Z","body":"I’m trying ‘make check’ now.\n\n> On Aug 31, 2015, at 1:18 PM, Don Bigler <dcb@energid.com> wrote:\n> \n> Correct, I modified the *setup.m script to load all dependent modules.  I did not run the test suite.  How do I run the test suite?  \n> \n>> On Aug 31, 2015, at 1:14 PM, Joel Andersson <notifications@github.com <mailto:notifications@github.com>> wrote:\n>> \n>> I see... I think I need to study closer how it actually works. There is a script that is run at initialization of a model. One could add to it a \"registration\" of the module to a MATLAB \"persistent\" variable, located in the SwigRef base class. I'll have another look and get back.\n>> \n>> In any case, do you have a solution to functions returning (owning) swig references? Did you try to run the test suite on your branch?\n>> \n>> —\n>> Reply to this email directly or view it on GitHub <https://github.com/jaeandersson/swig/pull/40#issuecomment-136432489>.\n>> \n> \n\n"}}