Update TODOs

Signed-off-by: Manoel Campos <manoelcampos@gmail.com>
cloudsimplus · Nov 4, 2022 · ee3429e · ee3429e
1 parent 5feeab6
commit ee3429e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/main/java/org/cloudbus/cloudsim/brokers/DatacenterBrokerAbstract.java b/src/main/java/org/cloudbus/cloudsim/brokers/DatacenterBrokerAbstract.java
@@ -664,7 +664,7 @@ private void notifyOnVmsCreatedListeners() {
             return;
         }
 
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onVmsCreatedListeners.size(); i++) {
             final var listener = onVmsCreatedListeners.get(i);
             listener.update(DatacenterBrokerEventInfo.of(listener, this));

diff --git a/src/main/java/org/cloudbus/cloudsim/vms/VmSimple.java b/src/main/java/org/cloudbus/cloudsim/vms/VmSimple.java
@@ -664,7 +664,7 @@ public final void setInMigration(final boolean migrating) {
      * @param targetHost the Host the VM is migrating to
      */
     public void updateMigrationStartListeners(final Host targetHost){
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onMigrationStartListeners.size(); i++) {
             final var listener = onMigrationStartListeners.get(i);
             listener.update(VmHostEventInfo.of(listener, this, targetHost));
@@ -676,7 +676,7 @@ public void updateMigrationStartListeners(final Host targetHost){
      * @param targetHost the Host the VM has just migrated to
      */
     public void updateMigrationFinishListeners(final Host targetHost){
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onMigrationFinishListeners.size(); i++) {
             final var listener = onMigrationFinishListeners.get(i);
             listener.update(VmHostEventInfo.of(listener, this, targetHost));
@@ -884,7 +884,7 @@ public boolean isDelayed() {
 
     @Override
     public void notifyOnHostAllocationListeners() {
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onHostAllocationListeners.size(); i++) {
             final var listener = onHostAllocationListeners.get(i);
             listener.update(VmHostEventInfo.of(listener, this));
@@ -894,7 +894,7 @@ public void notifyOnHostAllocationListeners() {
     @Override
     public void notifyOnHostDeallocationListeners(final Host deallocatedHost) {
         requireNonNull(deallocatedHost);
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onHostDeallocationListeners.size(); i++) {
             final var listener = onHostDeallocationListeners.get(i);
             listener.update(VmHostEventInfo.of(listener, this, deallocatedHost));
@@ -905,7 +905,7 @@ public void notifyOnHostDeallocationListeners(final Host deallocatedHost) {
      * Notifies all registered listeners when the processing of the Vm is updated in its {@link Host}.
      */
     public void notifyOnUpdateProcessingListeners() {
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onUpdateProcessingListeners.size(); i++) {
             final var listener = onUpdateProcessingListeners.get(i);
             listener.update(VmHostEventInfo.of(listener, this));
@@ -915,7 +915,7 @@ public void notifyOnUpdateProcessingListeners() {
     @Override
     public void notifyOnCreationFailureListeners(final Datacenter failedDatacenter) {
         requireNonNull(failedDatacenter);
-        //Uses indexed for to avoid ConcurrentModificationException
+        // TODO: Workaround - Uses indexed for to avoid ConcurrentModificationException
         for (int i = 0; i < onCreationFailureListeners.size(); i++) {
             final var listener = onCreationFailureListeners.get(i);
             listener.update(VmDatacenterEventInfo.of(listener, this, failedDatacenter));