C++ clean up in polybori interface

[jdemeyer]

Use the C++ features of Cython instead of using ccobject.h hacks.

The interface is significantly more complicated than some other C++ interfaces. There are also some Cython oddities that I have not investigated further. Therefore, the result may not be super clean, but it's certainly a large improvement over the current situation.

Depends on #23857
Depends on #21083

CC: @tscrim @kiwifb

Component: cython

Author: Jeroen Demeyer

Branch/Commit: feee399

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/23943

[jdemeyer]

Description changed:

--- 
+++ 
@@ -1 +1 @@
-
+Use the C++ features of Cython instead of using `ccobject.h` hacks.

[jdemeyer]

Branch: u/jdemeyer/ticket/23943

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

9ec349e

C++ clean up in polybori

[sagetrac-git]

Commit: 9ec349e

[jdemeyer]

comment:4

This is harder than some other library clean up ticket. In particular I do not understand why some things are wrapped as DefaultRinged<T> instead of just T.

For example, why

cdef cppclass PBSet "DefaultRinged<BooleSet> "

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

2314a54

C++ clean up in polybori

[sagetrac-git]

Changed commit from 9ec349e to 2314a54

[jdemeyer]

New commits:

2314a54

C++ clean up in polybori

[jdemeyer]

Changed dependencies from #23857 to #23857, #21083

[sagetrac-git]

Changed commit from 2314a54 to dcefde5