Enum ordinal can <0?

[Bricks-Man]

• I have searched the issues of this repository and believe that this is not a duplicate.
• [] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

• Dubbo version: master
• Operating System version: deepin 15.9
• Java version: 8

I want to know more about dubbo.And I found this code in module dubbo-metrics/dubbo-metrics-api/.../MetrcLevel

static {
        for (MetricLevel level : MetricLevel.values()) {
            if (level.ordinal() < 0) {
                throw new RuntimeException("MetricLevel can not < 0");
            }
        }
    }

I think this code is futility.Is it possible of level.ordinal() < 0?

[CrazyHZM]

level.ordinal() the initial value is 0

[Bricks-Man]

level.ordinal() the initial value is 0

I know it's init with 0, but can it < 0?

[CrazyHZM]

level.ordinal() the initial value is 0

I know it's init with 0, but can it < 0?

I agree with you, this code is futility.

[cvictory]

Please join us to commit a pull request to remove this code . huazhongming <notifications@github.com> 于2019年1月25日周五 上午11:40写道：

…

level.ordinal() the initial value is 0 I know it's init with 0, but can it < 0? I agree with you, this code is futility. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3339 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AD5mbA1t8HBZQXngIiTc_k1dAhJbRAQuks5vGnywgaJpZM4aSFoB> .

-- Best Regard! cvitory

[Bricks-Man]

Please join us to commit a pull request to remove this code . huazhongming notifications@github.com 于2019年1月25日周五 上午11:40写道：
…
level.ordinal() the initial value is 0 I know it's init with 0, but can it < 0? I agree with you, this code is futility. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3339 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AD5mbA1t8HBZQXngIiTc_k1dAhJbRAQuks5vGnywgaJpZM4aSFoB .
-- Best Regard! cvitory

I have committed a pr.

[kexianjun]

closed since this pr #3346 has been merged