rubinius/rubinius-processor — pull request #1 by jemc (2013-12-12 07:27:03+00:00)
Title: Keyword Arguments

This is a supporting pull request for https://github.com/rubinius/rubinius-melbourne/pull/5 , which is an attempt to address https://github.com/rubinius/rubinius/issues/2669

In the other pull request, I updated the melbourne parser to import MRI's support for keyword arguments. In this pull request, I updated `#process_args` to allow for the extra two arguments that the parser tries to pass to it. I also added `#process_kw_arg`, which currently handles named keyword arguments similarly to optional arguments (at least from rubinius-processor's standpoint).

Note that this change now requires the relevant changes to be propagated to the rubinius-ast gem (specifically, `FormalArguments19` must now accept the two new arguments as well, and actually handle them), and then possibly to other parts of the rubinius toolset.

When it comes to the changes in rubinius-ast (and beyond), we're reaching the point where it might be best for a rubinius core dev with a clearer vision of how they want to actually implement the feature to step in. However, in the absence of that, I'll take a look at it and keeping hacking away at this issue in any way I can.