Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename module LinkedLists to LinkedList or List #12711

Open
BryantLam opened this issue Mar 26, 2019 · 2 comments
Open

Rename module LinkedLists to LinkedList or List #12711

BryantLam opened this issue Mar 26, 2019 · 2 comments
Labels
area: Libraries / Modules type: Design

Comments

@BryantLam
Copy link

Related #6698

Somewhat sarcastically, why is it module LinkedLists? Is there more than one list in there? Should I anticipate a module Vectors? 😄

Rename LinkedLists to LinkedList or just List if you plan to support lists that aren't simply linked.
@e-kayrakli
Copy link
Contributor

I suspect this is the reason: #6979

@BryantLam
Copy link
Author

BryantLam commented Mar 26, 2019 •
edited
Loading

This is why I think anything Chapel-provided should be refactored into a std namespace.

I assume the std maintainers wouldn't clobber their own names and importing std.List is just as easy.

@BryantLam BryantLam mentioned this issue Mar 26, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: Libraries / Modules type: Design
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ben-albrecht @e-kayrakli @BryantLam