StdSet and StdMultiset support

[PraneethJain]

/~https://github.com/PraneethJain/libcxxwrap-julia#StdMultiset

Add support for std::set and std::multiset

[barche]

The reason for the failures on macOS is that you are building libcxxwrap-julia v0.12.2, but the latest JLL is at v0.12.3 and on mac the library version is stored in the referenced dylib. A simple rebase of your libcxxwrap-julia changes onto the latest main (which is ias 0.12.3) should fix this.

[PraneethJain]

Hey, the DOVERRIDE_VERSION_TO_JLL flag was making it so that the JLL version was being pulled from github releases, on which the latest was 0.12.2. I've turned it off so that the correct version is now built.

[barche]

I checked the StdSet part and that is fine for me. Optionally, you could try adding some prettier printing, I think this can be largely simplified by making set and multiset inherit from the AbstractSet Julia type (requires a change on the C++ side). It's probably best to have this inheritance structure for maximal Julia compatibility.

[PraneethJain]

I checked the StdSet part and that is fine for me. Optionally, you could try adding some prettier printing, I think this can be largely simplified by making set and multiset inherit from the AbstractSet Julia type (requires a change on the C++ side). It's probably best to have this inheritance structure for maximal Julia compatibility.

Inheriting from AbstractSet requires Base.iterate to be implemented for pretty printing, which depends on iterators. So changing the inheritance structure after the iterators PR is merged makes sense. Is that fine?

[barche]

Inheriting from AbstractSet requires Base.iterate to be implemented for pretty printing, which depends on iterators. So changing the inheritance structure after the iterators PR is merged makes sense. Is that fine?

Yes, that is OK!