Add support for directors

[jgillis]

[KrisThielemans]

thanks for this. I have a few compilation errors when running the test-suite. One of this is related to the fact that you're calling error() (3 times, e.g. in wrapConstructor). However, that's an Octave function that doesn't exist in MATLAB. I no longer get compilation errors when using

mexErrMsgIdAndTxt("SWIG:RuntimeError",errortext);

but I'm not 100% sure if this is what is needed. What do you think?

[KrisThielemans]

apologies. The code in wrapConstructor was fine. This error is only in MATLAB::MATLAB when writing to the director_prot_ctor_code (L272 of matlab.cxx)

[KrisThielemans]

I've now commited this and one other fix in my fork. The other fix adds some "const" to member functions. This now makes quite a lot more test cases compile (and a few more run...).

you can find it on

https://github.com/KrisThielemans/swig/tree/director_fixes

Sadly, I'm still not used to pull requests, so this branch also contains updates from Pull Request #41

[jaeandersson]

I've pulled your changes. @jgillis - close?

[KrisThielemans]

thanks! There's still quite a lot of failures in the test-suite related to directors. And @jgillis mentioned a memory leak. So I guess we cannot close this issue yet?

[jaeandersson]

Looks like directors now only work when there is exactly one output. I tried to use it with a function with void return and it failed.

[jaeandersson]

The director code really needs cleanup for maintainability.

[jgillis]

Design can be refactored using mxCalloc/mexMakeMemoryPersistent

[jaeandersson]

Interesting. Will you take a shot at it?

[KrisThielemans]

Most of the director support seems to work, but there are still 2 failures after #96.