Merge pull request #42882 from Shadow-Devil/add-override-annotation

Add @OverRide annotation everywhere
ballerina-platform · Jun 18, 2024 · eedd40d · eedd40d
2 parents 8e81723 + b6dfdaa
commit eedd40d
Show file tree

Hide file tree

Showing 479 changed files with 1,380 additions and 2 deletions.
diff --git a/ballerina-shell/modules/shell-core/src/main/java/io/ballerina/shell/EvaluatorImpl.java b/ballerina-shell/modules/shell-core/src/main/java/io/ballerina/shell/EvaluatorImpl.java
@@ -80,6 +80,7 @@ public String evaluate(String source) throws BallerinaShellException {
         }
     }
 
+    @Override
     public ShellCompilation getCompilation(String source) {
         PackageCompilation compilation;
         ExceptionStatus exceptionStatus;

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/FunctionType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/FunctionType.java
@@ -32,6 +32,7 @@ public interface FunctionType extends AnnotatableType {
 
     Type getReturnType();
 
+    @Override
     long getFlags();
 
     Type getReturnParameterType();

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/RemoteMethodType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/RemoteMethodType.java
@@ -24,8 +24,10 @@
  */
 public interface RemoteMethodType extends MethodType {
 
+    @Override
     ObjectType getParentObjectType();
 
+    @Override
     FunctionType getType();
 
 }
diff --git a/...ntime/src/main/java/io/ballerina/runtime/api/types/SelectivelyImmutableReferenceType.java b/...ntime/src/main/java/io/ballerina/runtime/api/types/SelectivelyImmutableReferenceType.java
@@ -25,7 +25,9 @@
  */
 public interface SelectivelyImmutableReferenceType extends IntersectableReferenceType {
 
+    @Override
     IntersectionType getImmutableType();
 
+    @Override
     void setImmutableType(IntersectionType immutableType);
 }
diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/StructureType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/StructureType.java
@@ -30,5 +30,6 @@ public interface StructureType extends AnnotatableType {
 
     Map<String, Field> getFields();
 
+    @Override
     long getFlags();
 }
diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/UnionType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/types/UnionType.java
@@ -32,5 +32,6 @@ public interface UnionType extends SelectivelyImmutableReferenceType {
 
     boolean isCyclic();
 
+    @Override
     long getFlags();
 }
diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BError.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BError.java
@@ -56,13 +56,15 @@ public BError(BString message) {
      *
      * @return error cause
      */
+    @Override
     public abstract BError getCause();
 
     /**
      * Print error stack trace to the given {@code PrintWriter}.
      *
      * @param printWriter {@code PrintWriter} to be used
      */
+    @Override
     public void printStackTrace(PrintWriter printWriter) {
         printWriter.print(ERROR_PRINT_PREFIX + getPrintableStackTrace());
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BMap.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BMap.java
@@ -146,6 +146,7 @@ public interface BMap<K, V> extends BRefValue, BCollection {
      *
      * @return the number of key-value mappings in this map
      */
+    @Override
     int size();
 
     /**

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BObject.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BObject.java
@@ -48,6 +48,7 @@ public interface BObject extends BRefValue {
      * @deprecated use {@link BObject#getOriginalType()} ()} instead.
      * The API {@link BValue#getType()} should be used after fixing the issue #39850.
      */
+    @Override
     @Deprecated
     ObjectType getType();
 

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BRefValue.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BRefValue.java
@@ -41,6 +41,7 @@ public interface BRefValue extends BValue {
      *
      * @return  A new copy of the value
      */
+    @Override
     Object copy(Map<Object, Object> refs);
 
     /**
@@ -51,6 +52,7 @@ public interface BRefValue extends BValue {
      *
      * @return  A new copy of the value
      */
+    @Override
     Object frozenCopy(Map<Object, Object> refs);
 
     /**

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BTable.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BTable.java
@@ -146,6 +146,7 @@ public interface BTable<K, V> extends BRefValue, BCollection {
      *
      * @return the number of key-value mappings in this map
      */
+    @Override
     int size();
 
     /**

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BXml.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/api/values/BXml.java
@@ -267,6 +267,7 @@ public interface BXml extends BRefValue, BCollection {
      * 
      * @return length of this XML sequence.
      */
+    @Override
     int size();
 
     /**

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalEnvironment.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalEnvironment.java
@@ -66,6 +66,7 @@ public BalEnvironment(Strand strand, Module currentModule, String funcName, Para
      *
      * @return function name
      */
+    @Override
     public String getFunctionName() {
         return funcName;
     }
@@ -75,6 +76,7 @@ public String getFunctionName() {
      *
      * @return array of {@link Parameter}
      */
+    @Override
     public Parameter[] getFunctionPathParameters() {
         return funcPathParams;
     }
@@ -87,6 +89,7 @@ public Parameter[] getFunctionPathParameters() {
      *
      * @return BalFuture which will resume the current strand when completed.
      */
+    @Override
     public BalFuture markAsync() {
         strand.blockedOnExtern = true;
         strand.setState(State.BLOCK_AND_YIELD);
@@ -98,6 +101,7 @@ public BalFuture markAsync() {
      *
      * @return Ballerina runtime instance.
      */
+    @Override
     public BalRuntime getRuntime() {
         return new BalRuntime(strand.scheduler, currentModule);
     }
@@ -107,6 +111,7 @@ public BalRuntime getRuntime() {
      *
      * @return module of the environment.
      */
+    @Override
     public Module getCurrentModule() {
         return currentModule;
     }
@@ -116,6 +121,7 @@ public Module getCurrentModule() {
      *
      * @return Strand id.
      */
+    @Override
     public int getStrandId() {
         return strand.getId();
     }
@@ -126,6 +132,7 @@ public int getStrandId() {
      *
      * @return Optional strand name.
      */
+    @Override
     public Optional<String> getStrandName() {
         return strand.getName();
     }
@@ -135,6 +142,7 @@ public Optional<String> getStrandName() {
      *
      * @return metadata of the strand.
      */
+    @Override
     public StrandMetadata getStrandMetadata() {
         return strand.getMetadata();
     }
@@ -145,6 +153,7 @@ public StrandMetadata getStrandMetadata() {
      * @param key   string key
      * @param value value to be store in the strand
      */
+    @Override
     public void setStrandLocal(String key, Object value) {
         strand.setProperty(key, value);
     }
@@ -155,10 +164,12 @@ public void setStrandLocal(String key, Object value) {
      * @param key key
      * @return value stored in the strand.
      */
+    @Override
     public Object getStrandLocal(String key) {
         return strand.getProperty(key);
     }
 
+    @Override
     public Repository getRepository() {
         return this.repository;
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalFuture.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalFuture.java
@@ -40,6 +40,7 @@ public BalFuture(Strand strand) {
         this.strand = strand;
     }
 
+    @Override
     public void complete(Object returnValue) {
         if (visited.getAndSet(true)) {
             throw ErrorCreator.createError(StringUtils.fromString("cannot complete the same future twice."),

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalRuntime.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/BalRuntime.java
@@ -77,26 +77,30 @@ public BalRuntime(Module module) {
         this.module = module;
     }
 
+    @Override
     public void init() {
         invokeConfigInit();
         invokeMethodAsync("$moduleInit", null, PredefinedTypes.TYPE_NULL, "init", new Object[1]);
         moduleInitialized = true;
     }
 
+    @Override
     public void start() {
         if (!moduleInitialized) {
             throw ErrorHelper.getRuntimeException(ErrorCodes.INVALID_METHOD_CALL, "start");
         }
         invokeMethodAsync("$moduleStart", null, PredefinedTypes.TYPE_NULL, "start", new Object[1]);
     }
 
+    @Override
     public void invokeMethodAsync(String functionName, Callback callback, Object... args) {
         if (!moduleInitialized) {
             throw ErrorHelper.getRuntimeException(ErrorCodes.INVALID_FUNCTION_INVOCATION, functionName);
         }
         invokeMethodAsync(functionName, callback, PredefinedTypes.TYPE_ANY, functionName, args);
     }
 
+    @Override
     public void stop() {
         if (!moduleInitialized) {
             throw ErrorHelper.getRuntimeException(ErrorCodes.INVALID_METHOD_CALL, "stop");
@@ -136,6 +140,7 @@ private void invokeMethodAsync(String functionName, Callback callback, Type retu
      * This method needs to be called if object.getType().isIsolated() or
      * object.getType().isIsolated(methodName) returns false.
      */
+    @Override
     public BFuture invokeMethodAsyncSequentially(BObject object, String methodName, String strandName,
                                                  StrandMetadata metadata,
                                                  Callback callback, Map<String, Object> properties,
@@ -183,6 +188,7 @@ public void notifyFailure(BError error) {
      * This method needs to be called if both object.getType().isIsolated() and
      * object.getType().isIsolated(methodName) returns true.
      */
+    @Override
     public BFuture invokeMethodAsyncConcurrently(BObject object, String methodName, String strandName,
                                                  StrandMetadata metadata,
                                                  Callback callback, Map<String, Object> properties,
@@ -236,6 +242,7 @@ public void notifyFailure(BError error) {
      * We can decide the object method isolation if and only if both object.getType().isIsolated() and
      * object.getType().isIsolated(methodName) returns true.
      */
+    @Override
     @Deprecated
     public BFuture invokeMethodAsync(BObject object, String methodName, String strandName, StrandMetadata metadata,
                                      Callback callback, Map<String, Object> properties,
@@ -288,6 +295,7 @@ public void notifyFailure(BError error) {
      * We can decide the object method isolation if both object.getType().isIsolated() and
      * object.getType().isIsolated(methodName) returns true.
      */
+    @Override
     @Deprecated
     public Object invokeMethodAsync(BObject object, String methodName, String strandName, StrandMetadata metadata,
                                     Callback callback, Object... args) {
@@ -304,14 +312,17 @@ private void validateArgs(BObject object, String methodName) {
         }
     }
 
+    @Override
     public void registerListener(BObject listener) {
         scheduler.getRuntimeRegistry().registerListener(listener);
     }
 
+    @Override
     public void deregisterListener(BObject listener) {
         scheduler.getRuntimeRegistry().deregisterListener(listener);
     }
 
+    @Override
     public void registerStopHandler(BFunctionPointer<?, ?> stopHandler) {
         scheduler.getRuntimeRegistry().registerStopHandler(stopHandler);
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/CycleUtils.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/CycleUtils.java
@@ -40,6 +40,7 @@ public Node(Object obj, BLink parent) {
             this.parent = parent;
         }
 
+        @Override
         public boolean hasCyclesSoFar() {
             Node parent = (Node) this.parent;
             while (parent != null) {

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/JsonGenerator.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/JsonGenerator.java
@@ -274,6 +274,7 @@ public void flush() throws IOException {
         this.writer.flush();
     }
 
+    @Override
     public void close() throws IOException {
         this.writer.close();
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/JsonParser.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/JsonParser.java
@@ -324,6 +324,7 @@ private static long convertToInt(Type targetType, String inputValue) throws Pars
             }
         }
 
+        @Override
         protected State finalizeObject() throws ParserException {
             Type targetType = this.targetTypes.get(this.targetTypes.size() - 1);
             switch (targetType.getTag()) {
@@ -448,6 +449,7 @@ private void processUnionTableFiniteType(Type targetType) {
             }
         }
 
+        @Override
         protected State initNewObject() throws ParserException {
             if (charBuffIndex != 0) {
                 throw new ParserException(UNRECOGNIZED_TOKEN +  "{'");
@@ -537,6 +539,7 @@ private void handleCurrentJsonNodeForObject() throws ParserException {
             }
         }
 
+        @Override
         protected State initNewArray() throws ParserException {
             if (charBuffIndex != 0) {
                 throw new ParserException(UNRECOGNIZED_TOKEN +  "['");
@@ -920,6 +923,7 @@ private void processNonStringValueAsJson(String str, ValueType type) throws Pars
             setValueToJsonType(type, getNonStringValueAsJson(str));
         }
 
+        @Override
         void processNonStringValue(ValueType type) throws ParserException {
             String str = value();
             Type targetType = this.targetTypes.get(this.targetTypes.size() - 1);
@@ -996,6 +1000,7 @@ private void processArrayType(String str, ArrayType referredType) throws ParserE
             this.listIndices.set(this.listIndices.size() - 1, listIndex + 1);
         }
 
+        @Override
         void setValueToJsonType(ValueType type, Object value) {
             switch (type) {
                 case ARRAY_ELEMENT:

diff --git a/...ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnnotatableType.java b/...ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnnotatableType.java
@@ -43,14 +43,17 @@ public void setAnnotations(BMap<BString, Object> annotations) {
         this.annotations = annotations;
     }
 
+    @Override
     public BMap<BString, Object> getAnnotations() {
         return (BMap<BString, Object>) this.annotations.copy(new LinkedHashMap<>());
     }
 
+    @Override
     public Object getAnnotation(BString key) {
         return this.annotations.get(key);
     }
 
+    @Override
     public Object getAnnotation(BString pkg, BString annotName) {
         return this.annotations.get(StringUtils.fromString(pkg.getValue() + ":" + annotName.getValue()));
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnyType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnyType.java
@@ -71,6 +71,7 @@ public int getTag() {
         return TypeTags.ANY_TAG;
     }
 
+    @Override
     public boolean isNilable() {
         return true;
     }

diff --git a/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnydataType.java b/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/types/BAnydataType.java
@@ -74,6 +74,7 @@ public int getTag() {
         return TypeTags.ANYDATA_TAG;
     }
 
+    @Override
     public boolean isNilable() {
         return true;
     }