FEAT: Support default arguments in functions

[0xe]

Adds support for default values in function parameters and default values in destructuring assignment.

What's not implemented yet, that I plan to implement subsequently:

• Destructuring with defaults in for loops (eg: for (let {x = 2} = {}; ; ))
• Reading [Symbol.iterator] from default array/object
• Nested Literals

(Added ignored tests for these in DefaultParametersTest)

[rbri]

@0xe wow great, i really need this feature and i will try to support you to get this out of the door.

[rbri]

@0xe maybe you can start simple only by supporting scalar values

and i think we have to place this behind a feature switch e.g. like this

[rbri]

because i like to work on this i removed the draft flag - hope this is ok

[rbri]

draft flag is back

[0xe]

Sorry, haven't had the chance to get back at this due to some personal commitments. Should be able to work again on this from next week.

[p-bakker]

Great to see your continued effort to implement this!

Think you're getting closer to this PR being ready for review and merge? Not trying to put pressure on or anything, just wondering

[0xe]

Great to see your continued effort to implement this!

Think you're getting closer to this PR being ready for review and merge? Not trying to put pressure on or anything, just wondering

Thank you! Yes, hopefully in a couple of days, I can mark it ready for review.

[0xe]

What's not implemented yet:

• Destructuring with defaults in for loops (eg: for (let {x = 2} = {}; ; ))
• Reading [Symbol.iterator] from default array/object
• Nested Literals

(Added ignored tests for these in DefaultParametersTest)

[gbrail]

This is looking good and a very helpful improvement judging from the number of tests that start to pass. Let us know when you have a chance for us to try it out more -- right now it needs another merge with master (sorry, things are actually moving sort of fast with this project!)

[0xe]

This is looking good and a very helpful improvement judging from the number of tests that start to pass. Let us know when you have a chance for us to try it out more -- right now it needs another merge with master (sorry, things are actually moving sort of fast with this project!)

Thank you. Updated with master.