Handle null function returned from CLion code

method is declared as @nullable in CLion SDK though sdkcompat ignores this hint. fixes bazelbuild#5247.
ujohnny · Aug 15, 2023 · d5fa54c · d5fa54c
1 parent dd0b5a5
commit d5fa54c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/sdkcompat/v231/com/google/idea/sdkcompat/clion/CMakeNotificationProviderWrapper.java b/sdkcompat/v231/com/google/idea/sdkcompat/clion/CMakeNotificationProviderWrapper.java
@@ -6,6 +6,7 @@
 import com.intellij.openapi.vfs.VirtualFile;
 import com.jetbrains.cidr.cpp.cmake.workspace.CMakeNotificationProvider;
 
+import javax.annotation.Nullable;
 import javax.swing.*;
 
 // #api223
@@ -16,7 +17,13 @@ public CMakeNotificationProviderWrapper(){
         this.value = new CMakeNotificationProvider();
     }
 
+    @Nullable
     public JComponent createNotificationPanel(VirtualFile virtualFile, FileEditor fileEditor, Project project) {
-        return this.value.collectNotificationData(project, virtualFile).apply(fileEditor);
+        var notificationProducer = this.value.collectNotificationData(project, virtualFile);
+        if (notificationProducer != null) {
+            return notificationProducer.apply(fileEditor);
+        }
+
+        return null;
     }
 }
diff --git a/sdkcompat/v232/com/google/idea/sdkcompat/clion/CMakeNotificationProviderWrapper.java b/sdkcompat/v232/com/google/idea/sdkcompat/clion/CMakeNotificationProviderWrapper.java
@@ -6,6 +6,7 @@
 import com.intellij.openapi.vfs.VirtualFile;
 import com.jetbrains.cidr.cpp.cmake.workspace.CMakeNotificationProvider;
 
+import javax.annotation.Nullable;
 import javax.swing.*;
 
 // #api223
@@ -16,7 +17,13 @@ public CMakeNotificationProviderWrapper(){
         this.value = new CMakeNotificationProvider();
     }
 
+    @Nullable
     public JComponent createNotificationPanel(VirtualFile virtualFile, FileEditor fileEditor, Project project) {
-        return this.value.collectNotificationData(project, virtualFile).apply(fileEditor);
+        var notificationProducer = this.value.collectNotificationData(project, virtualFile);
+        if (notificationProducer != null) {
+            return notificationProducer.apply(fileEditor);
+        }
+
+        return null;
     }
 }