Skip to content

Sevaral style fix for typelist.d#3238

Closed
ghost wants to merge 4 commits intomasterfrom
unknown repository
Closed

Sevaral style fix for typelist.d#3238
ghost wants to merge 4 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Apr 27, 2015

No description provided.

@jmdavis
Copy link
Member

jmdavis commented Apr 27, 2015

Wait, std.typelist is still in the repo? It's not in the makefile and was supposed to have been removed ages ago. AFAIK, it was never actually built as part of Phobos. So, doing work on it like this doesn't even make sense. Rather, it should just be deleted from the repo.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line actually is different from line 264. alias vs int. I'm not sure if that means it's necessary though.

@mihails-strasuns
Copy link

I was just about to delete this module as part of #3128 :)

@jmdavis jmdavis closed this Apr 27, 2015
@jmdavis
Copy link
Member

jmdavis commented Apr 27, 2015

@bbasile Well, thanks for taking the time to clean up std.typelist, but I'm afraid that it's wasted effort, since it's all dead code. Sorry.

@ghost ghost deleted the fixstyle_typelist branch April 28, 2015 02:39
@ghost ghost mentioned this pull request Aug 15, 2015
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.

3 participants