Forum: IronRuby Code Review: GenericMethodsAndOverloads

Announcement (2017-05-07): is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see and for other Rails- und Ruby-related community platforms.
Tomas M. (Guest)
on 2009-01-13 01:04
(Received via mailing list)
Attachment: GenericMethodsAndOverloads.diff (0 Bytes)
tfpt review "/shelveset:GenericMethodsAndOverloads;REDMOND\tomat"

    Implements generic methods parameters binding and explicit overload
selection. Adds methods Method/UnboundMethod#of and
Method/UnboundMethod#overloads. Method#of takes a list of Ruby classes
or CLR types and returns a Method instance that has bound generic
parameters to these classes/types. Method#overloads takes a list of Ruby
classes or CLR types and returns a Method instance that includes only
those CLR methods grouped in the Method object whose parameters are of
the given types.

Curt H. (Guest)
on 2009-01-14 06:49
(Received via mailing list)
Changes look good overall.

In RubyMethodGroupInfo.TryBindGenericParameters, an empty set of types
will return all methods in MethodBases whether or not they're generic.
What's the reason for this behavior?

There's a chunk of code that was added to Utils.cs that's indented too
Tomas M. (Guest)
on 2009-01-14 08:32
(Received via mailing list)
TryBindGenericParameters should look like:

            foreach (var method in MethodBases) {
                if (method.IsGenericMethodDefinition) {
                    if (typeArguments.Length ==
method.GetGenericArguments().Length) {
                        Debug.Assert(!(method is ConstructorInfo));
                } else if (typeArguments.Length == 0) {

Will include this fix in Indexers2 shelveset.

This topic is locked and can not be replied to.