Skip to content

Commit

Permalink
Use a more specific function type instead of Function object for Stri…
Browse files Browse the repository at this point in the history
…ng.prototype.replace callback.

PiperOrigin-RevId: 245856963
  • Loading branch information
jDramaix authored and copybara-github committed Apr 30, 2019
1 parent c27f562 commit bd2bb4a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
2 changes: 0 additions & 2 deletions java/elemental2/core/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ load("@com_google_jsinterop_generator//:jsinterop_generator.bzl", "jsinterop_gen
# TODO(b/36088887): Handle correctly @this annotation
# Removes non-existing static methods defined in Array type.
# TODO(b/36088884): clean-up the extern file directly
# Define better callback for String.replace.
# TODO(b/62258818): remove this specific patch when the bug is fixed and the extern file is updated.
# Make RegExp API stricter.
# TODO(b/38034193): clean up the RegExp API in the extern file directly.
patch_extern_file(
Expand Down
4 changes: 0 additions & 4 deletions java/elemental2/core/es3.js.diff
Original file line number Diff line number Diff line change
Expand Up @@ -202,10 +202,6 @@
< Array.some = function(arr, callback, opt_context) {};
<
< /**
1888c1773
< * @param {string|Function} replacement
---
> * @param {string|function(string, ...?):*} replacement
2050c1935
< * @param {*} pattern
---
Expand Down

0 comments on commit bd2bb4a

Please sign in to comment.