Remove unnecessary strictfp from PeepholeReplaceKnownMethods

lauraharker · copybara-github · commit 6635b9462b9f · 2025-09-02T14:15:25.000-07:00
Fixes console warnings such as:

src/com/google/javascript/jscomp/PeepholeReplaceKnownMethods.java:123: warning: [strictfp] as of release 17, all floating-point expressions are evaluated strictly and 'strictfp' is not requi
red
  private strictfp Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {
                        ^

PiperOrigin-RevId: 802287353
diff --git a/src/com/google/javascript/jscomp/PeepholeReplaceKnownMethods.java b/src/com/google/javascript/jscomp/PeepholeReplaceKnownMethods.java
@@ -120,7 +120,7 @@ private Node tryFoldKnownArrayMethods(Node subtree, Node callTarget) {
     return arraylit;
   }
 
-  private strictfp Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {
+  private Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {
     checkArgument(subtree.isCall() && callTarget.isGetProp());
     String methodName = callTarget.getString();
     switch (methodName) {
@@ -171,7 +171,7 @@ private strictfp Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {
   }
 
   /** Tries to evaluate a method on the Math object */
-  private strictfp Node tryFoldKnownMathMethods(Node subtree, Node callTarget) {
+  private Node tryFoldKnownMathMethods(Node subtree, Node callTarget) {
     checkArgument(subtree.isCall() && callTarget.isGetProp());
 
     // first collect the arguments, if they are all numbers then we proceed

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ private Node tryFoldKnownArrayMethods(Node subtree, Node callTarget) {`
`120`	`120`	`return arraylit;`
`121`	`121`	`}`
`122`	`122`
`123`		`- private strictfp Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {`
	`123`	`+ private Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {`
`124`	`124`	`checkArgument(subtree.isCall() && callTarget.isGetProp());`
`125`	`125`	`String methodName = callTarget.getString();`
`126`	`126`	`switch (methodName) {`
`@@ -171,7 +171,7 @@ private strictfp Node tryFoldKnownNumberMethods(Node subtree, Node callTarget) {`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`/** Tries to evaluate a method on the Math object */`
`174`		`- private strictfp Node tryFoldKnownMathMethods(Node subtree, Node callTarget) {`
	`174`	`+ private Node tryFoldKnownMathMethods(Node subtree, Node callTarget) {`
`175`	`175`	`checkArgument(subtree.isCall() && callTarget.isGetProp());`
`176`	`176`
`177`	`177`	`// first collect the arguments, if they are all numbers then we proceed`