diff options
| author |  Googler <noreply@google.com> | 2016-05-03 05:48:41 +0000 |
|---|---|---|
| committer |  Damien Martin-Guillerez <dmarting@google.com> | 2016-05-03 08:40:37 +0000 |
| commit | 08bd44c23ef2b5c7cc93828509bd4c50309afdbc (patch) | |
| tree | e7c0e027467e95fe36866141a74501a5d2610a6f /CONTRIBUTORS | |
| parent | a7fea8a1467a89adde75e67bc99bf0e040f71fe9 (diff) | |
Prioritize kwarg presence error in invokeObjectMethod.
I believe the original intention of this error check was to always fail if kwargs were present (while Skylark doesn't support kwargs, at least). However, the current code will most likely fail at findJavaMethod because only the args (i.e. not the kwargs) are used to find a matching method.
This issue has already been encountered on the bazel-discuss group (g/bazel-discuss/hHyf2txlS70).
With the proposed modification, I believe any kwargs provided to Skylark methods will cause the kwarg-specific error.
--
MOS_MIGRATED_REVID=121345719
Diffstat (limited to 'CONTRIBUTORS')
0 files changed, 0 insertions, 0 deletions