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.

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

Already on GitHub? Sign in to your account

Jump to bottom

RFE: -remove-element #97

Closed
fommil opened this issue Sep 10, 2014 · 4 comments
Closed

RFE: -remove-element #97

fommil opened this issue Sep 10, 2014 · 4 comments

Comments

@fommil
Copy link

fommil commented Sep 10, 2014

a remove element would be nice 馃槃 (synonym with delq, which I always forget the name of)
@fommil fommil changed the title RFE: -remove-el RFE: -remove-element Sep 10, 2014
@Fuco1
Copy link
Collaborator

Fuco1 commented Sep 10, 2014

I agree that a function like this would fit in the library, but -remove-element is a bit lengthy :( Ideally, it would be called just -remove but that is already taken.

Beware that delq compares with eq and that's a bit too specific, as it works "correctly" only for symbols. We would most likely want to remove by equal.

Right now, this can be implemented as (--remove (equal it ELEMENT) LIST). Maybe having a specific function wouldn't be so much of a gain after all. Let's see what others think.

@magnars
Copy link
Owner

magnars commented Sep 10, 2014

-remove-item perhaps?

@fommil
Copy link
Author

fommil commented Sep 10, 2014

I had thought remove-el(ement) originally but that clashes with el as in elisp

@bbatsov
Copy link
Contributor

bbatsov commented Dec 29, 2014

-remove-item sounds good to me.

@Fuco1 Fuco1 closed this as completed in cf7ca23 May 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@bbatsov @magnars @fommil @Fuco1