On 2022/12/7 2:37, Martin KaFai Lau wrote:
On 12/6/22 8:35 AM, Ji Rongfeng wrote:I have noticed that this patch has been marked as "Changes Requested" for a few days, but there's no comment so far, which is abnormal and confusing.It is obvious that a test is needed for this change.
Sure. These unique return values will be written into documentation and widely used in production environment. So I will add some tests, to make sure that they won't be changed accidentally in the future.