Attribute.CoreCLR, redundant check

[VasilievSerg]

The code:

public static Attribute? GetCustomAttribute(ParameterInfo element, Type attributeType, bool inherit)
{
    // ....
    Attribute[] attrib = GetCustomAttributes(element, attributeType, inherit);

    if (attrib == null || attrib.Length == 0) // <=
        return null;

    if (attrib.Length == 0) // <=
        return null;

    if (attrib.Length == 1)
        return attrib[0];

    throw new AmbiguousMatchException(SR.RFLCT_AmbigCust);
}

Link to the sources

The attrib.Length == 0 expression is checked twice. Perhaps it's not an error and just redundant code.
Could you check it?

[dotnet-issue-labeler]

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

[ghost]

Tagging subscribers to this area: @dotnet/area-system-runtime
See info in area-owners.md if you want to be subscribed.

Issue Details

---

The code:

public static Attribute? GetCustomAttribute(ParameterInfo element, Type attributeType, bool inherit)
{
    // ....
    Attribute[] attrib = GetCustomAttributes(element, attributeType, inherit);

    if (attrib == null || attrib.Length == 0) // <=
        return null;

    if (attrib.Length == 0) // <=
        return null;

    if (attrib.Length == 1)
        return attrib[0];

    throw new AmbiguousMatchException(SR.RFLCT_AmbigCust);
}

Link to the sources

The attrib.Length == 0 expression is checked twice. Perhaps it's not an error and just redundant code.
Could you check it?

Author:	VasilievSerg
Assignees:	-
Labels:	area-System.Runtime, untriaged
Milestone:	-