Thread overview
[Issue 4113] New: std.typetuple, std.typecons, TypeTuple, Tuple, tuple names
Sep 09, 2012
Denis Shelomovskij
Sep 09, 2012
timon.gehr@gmx.ch
April 21, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4113

           Summary: std.typetuple, std.typecons, TypeTuple, Tuple, tuple
                    names
           Product: D
           Version: future
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Phobos
        AssignedTo: nobody@puremagic.com
        ReportedBy: bearophile_hugs@eml.cc


--- Comment #0 from bearophile_hugs@eml.cc 2010-04-21 15:09:19 PDT ---
I think the Phobos module "std.typetuple" is not well named because the tuples can contain values (variables) too.

The std.typecons defines a Tuple. In D1 tuples are generally meant the ones used in std.typetuple. So I think this module too can be better named. Such tuples are implemented in D with structs, and in Pascal structs are named records.

So I propose the following five name changes:

std.typetuple ==> std.tuples
std.typetuple.TypeTuple ==> std.tuples.Tuple

std.typecons => std.records
std.typecons.Tuple ==> std.records.Record
std.typecons.tuple ==> std.records.record

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
September 09, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=4113


Denis Shelomovskij <verylonglogin.reg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |verylonglogin.reg@gmail.com


--- Comment #1 from Denis Shelomovskij <verylonglogin.reg@gmail.com> 2012-09-09 16:14:06 MSD ---
Some (a long) time ago, when I learned tuples, it was very confusing that `TypeTuple` is used as expression tuple. So I finally managed to do this boring renaming work and create https://github.com/D-Programming-Language/phobos/pull/780

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
September 09, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=4113



--- Comment #2 from bearophile_hugs@eml.cc 2012-09-09 07:01:17 PDT ---
(In reply to comment #1)

> https://github.com/D-Programming-Language/phobos/pull/780

Thank you. April 2010 when I have written this suggestion was fitter for name changes. The problem with similar suggestions is that if you leave them sleep in Bugzilla for years, they risk becoming "too much late to fix" things.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
September 09, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=4113


timon.gehr@gmx.ch changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |timon.gehr@gmx.ch


--- Comment #3 from timon.gehr@gmx.ch 2012-09-09 07:14:39 PDT ---
I usually just use template Seq(T...){ alias T Seq; }. The distinction between
type and expression tuples is not important. Anything that has 'tuple' in it is
a bad name because a) tuples usually don't get auto-expanded, b) we already
have
std.typecons.tuple. I object to renaming anything if both names are bad.

Ideally we'd have support for returning a Seq and there would also be a seq helper function akin to tuple.

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