Rename FutureWarning -> ParanoidWarning

Signed-off-by: Ben Sherman <bentshermann@gmail.com>

Author: bentsherman

modules/compiler/src/main/java/config/control/VariableScopeVisitor.java

@@ -241,7 +241,7 @@ public void visitVariableExpression(VariableExpression node) {
         Variable variable = vsc.findVariableDeclaration(name, node);
         if( variable == null ) {
             if( "it".equals(name) ) {
-                vsc.addFutureWarning("Implicit closure parameter `it` will not be supported in a future version", node);
+                vsc.addParanoidWarning("Implicit closure parameter `it` will not be supported in a future version", node);
             }
             else {
                 variable = new DynamicVariable(name, false);

modules/compiler/src/main/java/script/control/FutureWarning.java renamed to modules/compiler/src/main/java/script/control/ParanoidWarning.java

@@ -24,13 +24,13 @@
  *
  * @author Ben Sherman <bentshermann@gmail.com>
  */
-public class FutureWarning extends WarningMessage implements RelatedInformationAware {
+public class ParanoidWarning extends WarningMessage implements RelatedInformationAware {
 
     private String otherMessage;
 
     private ASTNode otherNode;
 
-    public FutureWarning(int importance, String message, CSTNode context, SourceUnit owner) {
+    public ParanoidWarning(int importance, String message, CSTNode context, SourceUnit owner) {
         super(importance, message, context, owner);
     }
 

modules/compiler/src/main/java/script/control/VariableScopeChecker.java

@@ -191,7 +191,7 @@ public Variable findDslMember(ClassNode cn, String name, ASTNode node) {
                 if( !name.equals(memberName) )
                     continue;
                 if( findAnnotation(mn, Deprecated.class).isPresent() )
-                    addFutureWarning("`" + name + "` is deprecated and will be removed in a future version", node);
+                    addParanoidWarning("`" + name + "` is deprecated and will be removed in a future version", node);
                 return wrapMethodAsVariable(mn, memberName);
             }
 
@@ -215,24 +215,24 @@ private Variable wrapMethodAsVariable(MethodNode mn, String name) {
         return pn;
     }
 
-    public void addFutureWarning(String message, String tokenText, ASTNode node, String otherMessage, ASTNode otherNode) {
+    public void addParanoidWarning(String message, String tokenText, ASTNode node, String otherMessage, ASTNode otherNode) {
         var token = new Token(0, tokenText, node.getLineNumber(), node.getColumnNumber()); // ASTNode to CSTNode
-        var warning = new FutureWarning(WarningMessage.POSSIBLE_ERRORS, message, token, sourceUnit);
+        var warning = new ParanoidWarning(WarningMessage.POSSIBLE_ERRORS, message, token, sourceUnit);
         if( otherNode != null )
             warning.setRelatedInformation(otherMessage, otherNode);
         sourceUnit.getErrorCollector().addWarning(warning);
     }
 
-    public void addFutureWarning(String message, ASTNode node, String otherMessage, ASTNode otherNode) {
-        addFutureWarning(message, "", node, otherMessage, otherNode);
+    public void addParanoidWarning(String message, ASTNode node, String otherMessage, ASTNode otherNode) {
+        addParanoidWarning(message, "", node, otherMessage, otherNode);
     }
 
-    public void addFutureWarning(String message, String tokenText, ASTNode node) {
-        addFutureWarning(message, tokenText, node, null, null);
+    public void addParanoidWarning(String message, String tokenText, ASTNode node) {
+        addParanoidWarning(message, tokenText, node, null, null);
     }
 
-    public void addFutureWarning(String message, ASTNode node) {
-        addFutureWarning(message, "", node, null, null);
+    public void addParanoidWarning(String message, ASTNode node) {
+        addParanoidWarning(message, "", node, null, null);
     }
 
     public void addError(String message, ASTNode node) {

modules/compiler/src/main/java/script/control/VariableScopeVisitor.java

@@ -158,7 +158,7 @@ public void visitFeatureFlag(FeatureFlagNode node) {
         if( result.isPresent() ) {
             var ffn = result.get();
             if( findAnnotation(ffn, Deprecated.class).isPresent() )
-                vsc.addFutureWarning("`" + node.name + "` is deprecated and will be removed in a future version", node.name, node);
+                vsc.addParanoidWarning("`" + node.name + "` is deprecated and will be removed in a future version", node.name, node);
             node.target = ffn;
         }
         else {
@@ -238,7 +238,7 @@ public void visitProcess(ProcessNode node) {
         vsc.popScope();
 
         if( !(node.when instanceof EmptyExpression) )
-            vsc.addFutureWarning("Process `when` section will not be supported in a future version", node.when);
+            vsc.addParanoidWarning("Process `when` section will not be supported in a future version", node.when);
         visit(node.when);
 
         visit(node.exec);
@@ -527,7 +527,7 @@ private void checkExternalWriteInAsyncClosure(VariableExpression target, Variabl
         var name = variable.getName();
         // TODO: should apply only to operator closures
         if( scope.isReferencedLocalVariable(name) && scope.getDeclaredVariable(name) == null )
-            vsc.addFutureWarning("Mutating an external variable in an operator closure may lead to a race condition", target, "External variable declared here", (ASTNode) variable);
+            vsc.addParanoidWarning("Mutating an external variable in an operator closure may lead to a race condition", target, "External variable declared here", (ASTNode) variable);
     }
 
     // expressions
@@ -648,13 +648,13 @@ public void visitVariableExpression(VariableExpression node) {
         Variable variable = vsc.findVariableDeclaration(name, node);
         if( variable == null ) {
             if( "it".equals(name) ) {
-                vsc.addFutureWarning("Implicit closure parameter `it` will not be supported in a future version", node);
+                vsc.addParanoidWarning("Implicit closure parameter `it` will not be supported in a future version", node);
             }
             else if( "args".equals(name) ) {
-                vsc.addFutureWarning("The use of `args` outside the entry workflow will not be supported in a future version", node);
+                vsc.addParanoidWarning("The use of `args` outside the entry workflow will not be supported in a future version", node);
             }
             else if( "params".equals(name) ) {
-                vsc.addFutureWarning("The use of `params` outside the entry workflow will not be supported in a future version", node);
+                vsc.addParanoidWarning("The use of `params` outside the entry workflow will not be supported in a future version", node);
             }
             else {
                 variable = new DynamicVariable(name, false);

modules/language-server/src/main/java/nextflow/lsp/services/LanguageService.java

@@ -40,7 +40,7 @@
 import nextflow.lsp.util.LanguageServerUtils;
 import nextflow.lsp.util.Logger;
 import nextflow.lsp.util.Positions;
-import nextflow.script.control.FutureWarning;
+import nextflow.script.control.ParanoidWarning;
 import nextflow.script.control.RelatedInformationAware;
 import nextflow.script.formatter.FormattingOptions;
 import org.codehaus.groovy.runtime.DefaultGroovyMethods;
@@ -395,7 +395,7 @@ protected void publishDiagnostics(Set<URI> changedUris) {
             }
 
             for( var warning : astCache.getWarnings(uri) ) {
-                if( !configuration.paranoidWarnings() && warning instanceof FutureWarning )
+                if( !configuration.paranoidWarnings() && warning instanceof ParanoidWarning )
                     continue;
 
                 var message = warning.getMessage();