<HTML><BODY><div>Hi, Sergey!</div><div>Thanks for the patch!</div><div>LGTM, except for a single nit below:</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><br>When `tonumber()` is recorded (as a part of a trace) for cdata argument<br>can't be converted to number the `nil` value is recorded as the yielded<br>result. But without special check on trace for cdata type this nil will<br>be returned for another type of cdata that can be converted.</div></blockquote><div>The first sentence lacks commas and is completely unreadable.</div><div>I suggest the following fix:</div><div>| When `tonumber()` is recorded (as a part of a trace) for a cdata argument that can't be converted to number, the `nil` value is | recorded as the yielded result.</div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div><br>This patch adds the corresponding check for recoding of failed cdata<br>conversions.</div></blockquote><div>Typo: s/recoding/recording</div><div> </div><div><snipped></div><div><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div><div> </div></BODY></HTML>