Skip to content

Delete typelist.d#3556

Closed
ghost wants to merge 1 commit intomasterfrom
unknown repository
Closed

Delete typelist.d#3556
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Aug 15, 2015

In #3238, @Dicebot told that it will be removed but it's still there (2.068).

@mihails-strasuns
Copy link

@WalterBright has prohibited removing it

@ghost
Copy link
Author

ghost commented Aug 15, 2015

I've proposed him to put it in undeaD

@jmdavis
Copy link
Member

jmdavis commented Aug 18, 2015

@WalterBright has prohibited removing it

AFAIK, it was never even compiled into Phobos. It's been in the repo but not in the makefiles. I don't see any reason why it would make any sense to keep it around. I don't see how any code could be using it.

@quickfur
Copy link
Member

@jmdavis So we should merge this then? Nothing in dmd/druntime/phobos references it, right?

@mihails-strasuns
Copy link

It was already merged once. Walter demanded it to be reverted and stay forever. Convince him first.

@jmdavis
Copy link
Member

jmdavis commented Aug 24, 2015

@quickfur If anything in Phobos ever referenced it, it was years ago. When I first learned about std.typelist, it was not in the Makefile, and it hasn't been since. I should have just deleted it then, but I wasn't that smart apparently. But if the repo history shows that std.typelist was never part of the Makefile, then it should be easy to convince Walter that it should just be deleted. That does require digging through the repo history though, which is likely to be a bit tedious.

@DmitryOlshansky
Copy link
Member

But if the repo history shows that std.typelist was never part of the Makefile, then it should be easy to convince Walter that it should just be deleted. That does require digging through the repo history though, which is likely to be a bit tedious.

It's a header-only meta module. Thing is I don't think anybody used it even back in 2007-2009, as the module was an experiment

@ghost
Copy link
Author

ghost commented Sep 12, 2015

the module that was intended to replace typelist is itself now renamed and deprecated.
std.typelist -> std.typetuple -> std.meta
You're not so punctilious with std.streams. By the way, it should be visible in the thread but since the beginning of this absurd discussion the move to undead is proposed:
dlang/undeaD#5

@ghost ghost closed this Sep 16, 2015
@ghost ghost deleted the BBasile-removetypelist branch September 23, 2015 22:51
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants