Add null guard for AddCommentToMethod and NumberToDuration (#6834)

davidh44 · web-flow · commit ba64ad5c0914 · 2026-03-31T22:18:46.000Z
diff --git a/v2-migration/src/main/java/software/amazon/awssdk/v2migration/AddCommentToMethod.java b/v2-migration/src/main/java/software/amazon/awssdk/v2migration/AddCommentToMethod.java
@@ -70,6 +70,9 @@ public String getDescription() {
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
+        if (methodPattern == null) {
+            return TreeVisitor.noop();
+        }
         return new Visitor(methodPattern, comment);
     }
 
diff --git a/v2-migration/src/main/java/software/amazon/awssdk/v2migration/NumberToDuration.java b/v2-migration/src/main/java/software/amazon/awssdk/v2migration/NumberToDuration.java
@@ -65,6 +65,9 @@ public String getDescription() {
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
+        if (methodPattern == null) {
+            return TreeVisitor.noop();
+        }
         return new Visitor(methodPattern, timeUnit);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,9 @@ public String getDescription() {`
`70`	`70`
`71`	`71`	`@Override`
`72`	`72`	`public TreeVisitor<?, ExecutionContext> getVisitor() {`
	`73`	`+ if (methodPattern == null) {`
	`74`	`+ return TreeVisitor.noop();`
	`75`	`+ }`
`73`	`76`	`return new Visitor(methodPattern, comment);`
`74`	`77`	`}`
`75`	`78`
Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,9 @@ public String getDescription() {`
`65`	`65`
`66`	`66`	`@Override`
`67`	`67`	`public TreeVisitor<?, ExecutionContext> getVisitor() {`
	`68`	`+ if (methodPattern == null) {`
	`69`	`+ return TreeVisitor.noop();`
	`70`	`+ }`
`68`	`71`	`return new Visitor(methodPattern, timeUnit);`
`69`	`72`	`}`
`70`	`73`