<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

  </head>

  <body>

    <div class="moz-cite-prefix">On 5/6/22 01:50, Jacek Caban wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:f5a7d579-9745-7296-61c3-65a9f894e140@codeweavers.com">

      <blockquote type="cite" style="color: #007cff;">Honestly, I don't

        particularly care about toString, it's just the only builtin I

        could think off the top of my head that could be tested for

        this. Because right now we handle it in exec_source, which is

        only for interpreted functions... but it's a good test case of

        such behavior.

        <br>

        <br>

        Do you think a fix like in 2/2 is more appropriate (also for

        quirks modes, but with null instead of undefined)? And perhaps

        fix whatever is causing toString to fail, which seems unrelated

        I guess.

        <br>

        <br>

        That might mean moving it out of exec_source though. </blockquote>

      <br>

      <br>

      Same story there, why do you think it's somehow related to

      detached scopes? The attached test fails with your patch.

    </blockquote>

    <p><br>

    </p>

    <p>I attached a wrong test, here is the right one.</p>

    <p><br>

    </p>

    <p>Jacek<br>

    </p>

  </body>

</html>