unnecessary compassion in b2_collision.cpp

[kritma]

    if (count < 3 || count > b2_maxPolygonVertices)
    {
        // check your data
        return hull;
    }

    count = b2Min(count, b2_maxPolygonVertices); //this line is unnecessary

[krabhi1]

How?

[HughPH]

count must always be less than or equal to b2_maxPolygonVertices according to the preceding if statement, so the call to b2Min is not required.

[kritma]

@krabhi7 this is same as

if(count < 3) {
    return hull;
}

if(count > b2_maxPolygonVertices) {
    return hull;
}

count = b2Min(count, b2_maxPolygonVertices); // count <= b2_maxPolygonVertices because of early return