[webkit-changes] [WebKit/WebKit] 81d93d: Set numBoundArgs after setting boundArgs

Rose noreply at github.com
Tue May 16 01:31:37 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 81d93da03d83c8e0e7d641030972ee2cf7c3c798
      https://github.com/WebKit/WebKit/commit/81d93da03d83c8e0e7d641030972ee2cf7c3c798
  Author: Rose <83477269+AtariDreams at users.noreply.github.com>
  Date:   2023-05-16 (Tue, 16 May 2023)

  Changed paths:
    M Source/JavaScriptCore/runtime/FunctionPrototype.cpp

  Log Message:
  -----------
  Set numBoundArgs after setting boundArgs
https://bugs.webkit.org/show_bug.cgi?id=255540

Reviewed by Yusuke Suzuki.

numBoundArgs is always zero because it is never changed
after being set to zero. It is clear from the code itself that the
number of arguments is meant to be set, but whoever programmed this part
forgot to do so.

* Source/JavaScriptCore/runtime/FunctionPrototype.cpp:
  (JSC::JSC_DEFINE_HOST_FUNCTION): Assign numBoundArgs to the size
  of the boundArgs array.

Canonical link: https://commits.webkit.org/264109@main




More information about the webkit-changes mailing list