<HTML><BODY><div>Hi, Sergey!</div><div>Thanks for the patch!</div><div>LGTM</div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div></div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16740732221088312708_BODY">When the profiling interval is parsed, `tvisnum()` is used to check that<br>the given value is a number. But in case of dual-number mode this check<br>returns false as far as the given value is an integer, so `tvisnumber()`<br>should be used to cover both cases.<br>---<br> src/lib_misc.c | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/src/lib_misc.c b/src/lib_misc.c<br>index 06fb9f9f..c18d297e 100644<br>--- a/src/lib_misc.c<br>+++ b/src/lib_misc.c<br>@@ -209,7 +209,7 @@ static int parse_sysprof_opts(lua_State *L, struct luam_Sysprof_Options *opt, in<br> {<br> cTValue *interval = lj_tab_getstr(options, lj_str_newlit(L, "interval"));<br> opt->interval = SYSPROF_DEFAULT_INTERVAL;<br>- if (interval && tvisnum(interval)) {<br>+ if (interval && tvisnumber(interval)) {<br> int32_t signed_interval = numberVint(interval);<br> if (signed_interval < 1)<br> return PROFILE_ERRUSE;<br>--<br>2.34.1</div></div></div></div></blockquote><div> </div></div></blockquote></BODY></HTML>