Confuse on the logic judge isRandomPort and the comment note | 随机端口判断的逻辑和注释容易混淆

[jnan806]

Describe the bug

file-location : sofa-rpc/core/api/src/main/java/com/alipay/sofa/rpc/common/utils/NetUtils.java function : public static boolean isRandomPort(int port)

logic in isRandomPort() is conflict with the comment, so which is the correct ？ the logic or the comment ?

Expected behavior

Actual behavior

Steps to reproduce

Minimal yet complete reproducer code (or GitHub URL to code)

Environment

• SOFARPC version:
• JVM version (e.g. java -version):
• OS version (e.g. uname -a):
• Maven version:
• IDE version:

[EvenLjj]

@jnan806 A precondition is that the port is between - 1 and 65535. Therefore, the comment that is less than 0 actually refers to - 1 too. Look very carefullyl. If you are interested, you can submit PR to modify this comment.

[jnan806]

@jnan806 A precondition is that the port is between - 1 and 65535. Therefore, the comment that is less than 0 actually refers to - 1 too. Look very carefullyl. If you are interested, you can submit PR to modify this comment.

glad to contribute it. Please assign it to me ?

[EvenLjj]

@jnan806 Okay, look forward to more participation.

[jnan806]

@EvenLjj PR has been submitted. And need an approval to run the workflow.