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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md #105

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

kamil-kielczewski
Copy link
Contributor

create multi-arguments functions

create multi-arguments functions
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@hazzik
Copy link
Contributor

hazzik commented Aug 30, 2020

If you want even more arguments for less characters you can do following:

"return(" + []["slice"]["call"](false+"") + ")=>f+t"

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@aemkei
Copy link
Owner

aemkei commented Aug 31, 2020

Well, that is a nice hack to allow multiple args.

Fixes - typos and
I change map( (f,t)=>f+t ) to map( (x,i)=>x+i ) to more traditional/redable version (because here I want to show technique without focus of size-code optimalization
In regexp part I use this technique which simplify this task, but I left old way because in some cases I think it can be useful
change fill to flat (which gives smaller size and not decrease readability)
add info about optimization
@kamil-kielczewski
Copy link
Contributor Author

kamil-kielczewski commented Aug 31, 2020

@hazzik Finally I add info about optimization trick which is very clever

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.

None yet

3 participants