> On Apr 24, 2026, at 11:15, Fujii Masao <[email protected]> wrote:
>
> On Thu, Apr 23, 2026 at 11:03 AM Chao Li <[email protected]> wrote:
>> I just checked old branches. Looks like 0001 can be back-patched down to
>> v10. Yes, pre-19, all branches use microsecond, so the back-patch should
>> change “ms” to “us”.
>
> Thanks! So I've pushed this unit change to master and backpatched it to
> all supported branches.
>
>
>> For v16 to v18, we can make a tiny improvement by replacing “1e9” with a
>> constant macro NS_PER_S. This change has been included in the diff.
>
> This change looks good to me. However, we should generally keep changes to
> old stable branches to the minimum required, so I applied this change only
> to master with 0002 patch.
>
>
>> I see 0002 a bit differently. In v19, the unit changed from microseconds to
>> nanoseconds, which introduced a potential overflow: nanoseconds require
>> int64, but the local variable remained int32. So I think this is actually a
>> v19-only bug.
>
> OK, I've pushed 0002 patch. Thanks!
>
> Regards,
>
> --
> Fujii Masao
Hi Fujii-san, thank you very much for pushing.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
