pySBOL remove() method causes Python to crash

[kirichoi]

It seems like remove() method is crashing Python. Python unittest tests remove functions by executing something like

doc.sequences.remove(0)

and assert an error to be raised when the object is assessed. Has there been any changes to the function making the above expression invalid?

[bbartley]

Yes, I introduced the bug. It's trying to delete a pointer twice. I'll look into it.

[bbartley]

The error occurs only in Python, not libSBOL, and is introduced in libsbol.i line 174. This is one of those annoying SWIG memory management issues, and will take some time to sort out. For now I've commented the line out (see 4884ff5), and the test should pass. (It will result in a memory leak when calling the remove method, however.)

[bbartley]

Related to #140