Code Review: DMO1

tfpt review “/shelveset:DMO1;REDMOND\tomat”
Comment :

  • Checks for an invalid number of arguments is passed to an attribute
    setter/getter or struct field setter/getter.
  • Implements correct splatting of arguments to Delegate constructor
    and enables previously failing tests.
  • Refactors RubyMethodGroupBase in preparation for obsolete API usage
  • Fixes a comment and adds OBSOLETE comments to obsolete API of
    MethodBinder and BindingTarget classes.


In RubyClass.cs, there’s an incorrect parameter passed to

var actualArgs = RubyMethodGroupBase.NormalizeArguments(metaBuilder,
args, SelfCallConvention.NoSelf, false, false);
if (actualArgs.Length == 1) {

} else {
metaBuilder.SetWrongNumberOfArgumentsError(actualArgs.Length, 0);
<-- should be “1”.

Looks good otherwise.

Good catch. Will fix it in the next changeset.


This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs