Bug 5699 – opBinary used to implement "in" in TotalContainer

The TotalContainer example class uses opBinary to implement the "in" operator, while it should use opBinaryRight.

Please be more specific. Where is this example?

I think it was on the example container for std.container that now is simply a table. I think this bug is obsolete. See http://www.digitalmars.com/d/2.0/phobos/std_container.html There is no longer a TotalContainer object.

I was referring to the TotalContainer class in std.container; you'll find it in the repo, here: https://github.com/D-Programming-Language/phobos/blob/master/std/container.d The bug is still there. The docs say, "The following documentation and type $(D TotalContainer) are intended for developers only," which might explain why it's not in the public docs -- it's a no-op dummy container, after all (an example to use as a starting-point). Still, if it is to serve as an example for developers, it should be correct.

Oh, my mistake! The original docs did show the container in the documentation, which is why I thought it was removed. The exact location is here: https://github.com/D-Programming-Language/phobos/blob/31d47df17fc77a5a16e4e51267eaa6952ba01213/std/container.d#L480

Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/4229d024ca9c7fc2c750cd60e94cd1cacb134016 Fix Issue 5699 - TotalContainer opBinaryRight for 'in' https://d.puremagic.com/issues/show_bug.cgi?id=5699 https://github.com/D-Programming-Language/phobos/commit/cc09f11d05962edf91fc1934700a2b6a8d7c97a1 Merge pull request #1947 from Poita/bug5699 Fix Issue 5699 - TotalContainer opBinaryRight for 'in'