Skip to content
New issue

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

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

Already on GitHub? # to your account

Rename e{objc,swift} to {objc,swift}e #141

Closed
wants to merge 1 commit into from
Closed

Conversation

kastiglione
Copy link
Contributor

In #121, two new chisel commands were added eobjc and eswift, for explicitly evaluating with a specific language. However this impacts the sanctity of the e command in lldb 😜. Sure I can from now on use ex where I previously used e, but we could also change the convention of how these commands are named.

@kastiglione
Copy link
Contributor Author

Thoughts @ultramiraculous @tomaskraina?

@ultramiraculous
Copy link

Yeah...I had a coworker grumble about this the other day. My complaint here would be that I've personally gotten used to es/eo.

Nobody wants to double down on this and just define e as a separate command...right? 😜

@kastiglione
Copy link
Contributor Author

Nobody wants to double down on this and just define e as a separate command...right? 😜

hmm, I guess that's possible too…

@kastiglione kastiglione closed this May 4, 2016
@kastiglione kastiglione deleted the leave-e-alone branch May 4, 2016 22:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants