8322890: Directly return in OldPLABSizeConstraintFunc

LizBing · albertnetymk · commit 886386c0396d · 2024-01-09T15:52:39.000Z
Reviewed-by: ayang, tschatzl
diff --git a/src/hotspot/share/gc/shared/jvmFlagConstraintsGC.cpp b/src/hotspot/share/gc/shared/jvmFlagConstraintsGC.cpp
@@ -93,13 +93,7 @@ JVMFlag::Error YoungPLABSizeConstraintFunc(size_t value, bool verbose) {
 }
 
 JVMFlag::Error OldPLABSizeConstraintFunc(size_t value, bool verbose) {
-  JVMFlag::Error status = JVMFlag::SUCCESS;
-
-  {
-    status = MinMaxPLABSizeBounds("OldPLABSize", value, verbose);
-  }
-
-  return status;
+  return MinMaxPLABSizeBounds("OldPLABSize", value, verbose);
 }
 
 JVMFlag::Error MinHeapFreeRatioConstraintFunc(uintx value, bool verbose) {

Original file line number	Diff line number	Diff line change
`@@ -93,13 +93,7 @@ JVMFlag::Error YoungPLABSizeConstraintFunc(size_t value, bool verbose) {`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`JVMFlag::Error OldPLABSizeConstraintFunc(size_t value, bool verbose) {`
`96`		`- JVMFlag::Error status = JVMFlag::SUCCESS;`
`97`		`-`
`98`		`- {`
`99`		`- status = MinMaxPLABSizeBounds("OldPLABSize", value, verbose);`
`100`		`- }`
`101`		`-`
`102`		`- return status;`
	`96`	`+ return MinMaxPLABSizeBounds("OldPLABSize", value, verbose);`
`103`	`97`	`}`
`104`	`98`
`105`	`99`	`JVMFlag::Error MinHeapFreeRatioConstraintFunc(uintx value, bool verbose) {`