Thread overview
[Issue 5699] New: opBinary used to implement "in" in TotalContainer
Mar 04, 2011
Magnus Lie Hetland
Mar 04, 2011
Jonathan M Davis
Mar 04, 2011
Magnus Lie Hetland
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699

           Summary: opBinary used to implement "in" in TotalContainer
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Mac OS X
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody@puremagic.com
        ReportedBy: magnus@hetland.org


--- Comment #0 from Magnus Lie Hetland <magnus@hetland.org> 2011-03-04 10:37:53 PST ---
The TotalContainer example class uses opBinary to implement the "in" operator, while it should use opBinaryRight.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699


Jonathan M Davis <jmdavisProg@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jmdavisProg@gmx.com


--- Comment #1 from Jonathan M Davis <jmdavisProg@gmx.com> 2011-03-04 11:23:15 PST ---
Please be more specific. Where is this example?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699


Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |schveiguy@yahoo.com
         Resolution|                            |INVALID


--- Comment #2 from Steven Schveighoffer <schveiguy@yahoo.com> 2011-03-04 11:33:53 PST ---
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.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699


Magnus Lie Hetland <magnus@hetland.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |


--- Comment #3 from Magnus Lie Hetland <magnus@hetland.org> 2011-03-04 11:58:09 PST ---
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.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699



--- Comment #4 from Steven Schveighoffer <schveiguy@yahoo.com> 2011-03-04 12:07:18 PST ---
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

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 04, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=5699


Andrei Alexandrescu <andrei@metalanguage.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |ASSIGNED
                 CC|                            |andrei@metalanguage.com
         AssignedTo|nobody@puremagic.com        |andrei@metalanguage.com


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------