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

drop optional args #1

Closed
juliangruber opened this issue Jan 20, 2013 · 2 comments
Closed

drop optional args #1

juliangruber opened this issue Jan 20, 2013 · 2 comments

Comments

@juliangruber
Copy link
Member

Can we make optional arguments really optional? Like options and callbacks?

In C that's not hard and the performance loss should be subliminal. The implementation will grow a but but it will be way more comfortable to use.
@rvagg
Copy link
Member

rvagg commented Jan 20, 2013

absolutely, that'd be the first thing I'd do. I'll put in a PR on the README when I get a chance to show what I'd make optional; unless someone else beats me to it.

ralphtheninja pushed a commit that referenced this issue Feb 16, 2013
Merge pull request #1 from rvagg/ralphtheninja-master
ec95dd0 
code style tweaks
@rvagg
Copy link
Member

rvagg commented Mar 4, 2013

done

@rvagg rvagg closed this as completed Mar 4, 2013
@19h 19h mentioned this issue Aug 24, 2013
Closed
@abliss abliss mentioned this issue Jun 15, 2014
Closed
@chjj chjj mentioned this issue Mar 7, 2016
Closed
@SatyajitKumar SatyajitKumar mentioned this issue Jul 21, 2016
Closed
peakji added a commit that referenced this issue Dec 18, 2016
@peakji
Merge pull request #1 from peakji/seek-reset-state
490d5a0 
Seek reset state
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
2 participants
@juliangruber @rvagg