So was the fix to iterate or use no dupes?
In those cases, iterate. Almost everywhere else became non-dups, so will now assert if you try to insert a duplicate...
Please Login to reply.
Oh, congrats on finding the bug btw. One less thing….
Also, thanks for the transparency/humility. It’s nice to know that accomplished developers are human too.
Do the asserts compile out in release mode? Should a return value be checked instead/also?