(groovy) branch master updated: Avoid potential NPE

[sunlan]

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 467e317561 Avoid potential NPE
467e317561 is described below

commit 467e317561fe667e5838fda7cdc118cb2f2d73d6
Author: Daniel Sun <sun...@apache.org>
AuthorDate: Sat Dec 28 23:15:23 2024 +0900

    Avoid potential NPE
---
 .../src/main/groovy/groovy/jmx/builder/JmxMetaMapBuilder.groovy         | 2 +-
 .../src/main/groovy/groovy/typecheckers/FormatStringChecker.groovy      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/subprojects/groovy-jmx/src/main/groovy/groovy/jmx/builder/JmxMetaMapBuilder.groovy
 
b/subprojects/groovy-jmx/src/main/groovy/groovy/jmx/builder/JmxMetaMapBuilder.groovy
index 38950e5978..f53de42889 100644
--- 
a/subprojects/groovy-jmx/src/main/groovy/groovy/jmx/builder/JmxMetaMapBuilder.groovy
+++ 
b/subprojects/groovy-jmx/src/main/groovy/groovy/jmx/builder/JmxMetaMapBuilder.groovy
@@ -467,7 +467,7 @@ class JmxMetaMapBuilder {
                     if (descriptor && descriptor instanceof Map) {
                         // foo:[params:["paramTypeName0":[name:"",desc:""], 
paramTypeNameN[:]]]
                         if (descriptor.params && descriptor.params instanceof 
Map) {
-                            params = descriptor?.params.keySet().toList()
+                            params = descriptor?.params?.keySet()?.toList()
                         }
                         // foo:[params:["paramTypeName0",...,"paramTypeNameN"]]
                         if (descriptor.params && descriptor.params instanceof 
List) {
diff --git 
a/subprojects/groovy-typecheckers/src/main/groovy/groovy/typecheckers/FormatStringChecker.groovy
 
b/subprojects/groovy-typecheckers/src/main/groovy/groovy/typecheckers/FormatStringChecker.groovy
index 0e08631280..893cab1ae0 100644
--- 
a/subprojects/groovy-typecheckers/src/main/groovy/groovy/typecheckers/FormatStringChecker.groovy
+++ 
b/subprojects/groovy-typecheckers/src/main/groovy/groovy/typecheckers/FormatStringChecker.groovy
@@ -255,7 +255,7 @@ class FormatStringChecker extends 
GroovyTypeCheckingExtensionSupport.TypeCheckin
             }
 
             void checkBadFlags(flagList, conversion, Expression target, String 
badFlags) {
-                def mismatched = flagList?.findAll { badFlags.contains(it) 
}.join()
+                def mismatched = flagList?.findAll { badFlags.contains(it) 
}?.join()
                 if (mismatched) {
                     addStaticTypeError("FormatFlagsConversionMismatch: 
Conversion = $conversion, Flags = '$mismatched'", target)
                 }