www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 4363] New: std.algorithm.Until is not a forward range

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363

           Summary: std.algorithm.Until is not a forward range
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: philippe.sigaud gmail.com



10:56:46 CEST ---
As of 2.047, to be a forward range one has to have a .save() method.

std.algorithm.Until does not have one, but should be a forward range.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 22 2010
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yebblies gmail.com



I've also noticed randomCover has the same problem.  Maybe rather than create a
separate bug report this one could list the ranges in phobos that need a
' property typeof(this) save() { return this; }' added?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 22 2010
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363




21:43:43 CEST ---


Yeah, I meant to scan std.algo but one of my daughters got sick and... well.

You could list other ranges that have the same problem, but in this case I need
to change the bug report's title.

...

OK, done. Go wild.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 23 2010
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Some std.algorithm ranges   |Some phobos ranges are not
                   |are not forward ranges (but |forward ranges (but should
                   |should be)                  |be)



std.numeric: GapWeightedSimilarityIncremental
std.random: LinearCongruentialEngine, MersenneTwisterEngine, RandomCover,
RandomSample
std.range: Radial, Repeat, Cycle, Recurrence, FrontTransversal, Transversal
std.regex: Regex
std.regexp: Splitter
std.algorithm: Filter, Splitter, Uniq, Group, Until, SetUnion, SetIntersection,
SetDifference, SetSymmetricDifference
std.container: SList(T).Range

This list is of any structs that define popFront and do not define save, and
seem to support copying.  I've excluded any types with allocated storage,
unless they define a copy constructor.  I've also excluded any that are
obviously designed to be input ranges (have a File handle etc) but a couple may
have slipped through.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 23 2010
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363




Changed the title to reflect that this report covers all of phobos, not just
std.algorithm.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 23 2010
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363




According to David Simcha's commcent in bug 4362, repeat and cycle have already
been fixed in svn.  This leaves:

std.numeric: GapWeightedSimilarityIncremental
std.random: LinearCongruentialEngine, MersenneTwisterEngine, RandomCover,
RandomSample
std.range: Radial, Recurrence, FrontTransversal, Transversal
std.regex: Regex
std.regexp: Splitter
std.algorithm: Filter, Splitter, Uniq, Group, Until, SetUnion, SetIntersection,
SetDifference, SetSymmetricDifference
std.container: SList(T).Range

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 24 2010
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Jesse.K.Phillips+D gmail.co
                   |                            |m



*** Issue 4467 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jul 15 2010
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=4363


David Simcha <dsimcha yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |dsimcha yahoo.com
         Resolution|                            |FIXED



Fixed SVN.  (At least I think I got all of them.)

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