swig

swig copied to clipboard

Originally mentioned in https://github.com/jaeandersson/swig/pull/96#issuecomment-998434304 I get a ton of warnings like ``` Warning: function ./+li_math/atan.m shadows a built-in function ``` This is likely due to https://savannah.gnu.org/bugs/?46849. There doesn't seem to...

KrisThielemans

`unions.ctest` and `imports.multicpptest` fail as Octave doesn't have `import` yet. It seems that these were overlooked in https://github.com/jaeandersson/swig/commit/b5e75d1891a64000508b511e220483b0547f5428

KrisThielemans

#96 reinstated the `subsref` functions in `Swigref.m`. However, when running the test-suite with recent Octave, it generates lots of warnings like ``` warning: 'uint64 matrix' object indexed with empty index...

KrisThielemans

Explicitly print a warning about missing support for save/load

4

I got some feedback from confused users of my SWIG-wrapped library that saved (with "save") and loaded back (with "load") their workspace containing SWIG Classes. Clearly the "save" and "load"...

traversaro

Fix unit tests (was: Broken unit tests on OS X)

39

jaeandersson

``` swig -Wall -python -c++ .\import_nomodule.i .\import_nomodule.i(37) : Warning 401: Base class 'Foo' ignored - unknown module name for base. Either import the appropriate module interface file or specify the...

KrisThielemans

https://github.com/jaeandersson/swig/blob/cd3f6c5fe9ed273fcc981dcae3e8b50d51529664/Lib/swig.swg#L426-L428 needs to be changed for MATLAB, as it doesn't allow functions starting with `_`. This creates an error in the test-suite ``` ERROR: Error: File: /home/kthielem/devel/build/swig/Examples/test-suite/matlab/+smart_pointer_member/Bar.m Line: 17 Column:...

KrisThielemans

Add support for directors

10

jgillis

Test suite broken

5

The test suite got (more) broken in one of the more recent commits.

jaeandersson

Package name vs. module name

2

The proxy classes and functions belonging to a module `mymodule` are currently placed in a directory `+mypackage`. Often `mymodule` and `mypackage` are the same, but this doesn't need to be...

jaeandersson