Merge pull request #1702 from pguyot/w24/remove-todos

Remove TODOs for things that have been done

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Author: bettio

src/libAtomVM/opcodesswitch.h

@@ -2605,7 +2605,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: implement wait/1
             case OP_WAIT: {
                 uint32_t label;
                 DECODE_LABEL(label, pc)
@@ -2622,7 +2621,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: implement wait_timeout/2
             case OP_WAIT_TIMEOUT: {
                 #ifdef IMPL_EXECUTE_LOOP
                     // PC for wait_timeout_trap_handler, just before label
@@ -2936,7 +2934,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 #ifdef IMPL_EXECUTE_LOOP
                     TRACE("is_number/2, label=%i, arg1=%lx\n", label, arg1);
 
-                    //TODO: check for floats too
                     if (!term_is_number(arg1)) {
                         pc = mod->labels[label];
                     }
@@ -3278,7 +3275,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                         #endif
 
                         #ifdef IMPL_EXECUTE_LOOP
-                            //TODO: check if src_value is a tuple
                             if (!jump_to_address && ((uint32_t) arity == cmp_value)) {
                                 jump_to_address = mod->labels[jmp_label];
                             }
@@ -3707,7 +3703,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
 
                 #ifdef IMPL_EXECUTE_LOOP
                     term catch_term = term_from_catch_label(mod->module_index, label);
-                    //TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, catch_term);
                 #endif
                 break;
@@ -3720,7 +3715,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 TRACE("try_end/1, reg=%c%i\n", T_DEST_REG(dreg));
 
                 #ifdef IMPL_EXECUTE_LOOP
-                    //TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, term_nil());
                 #endif
                 break;
@@ -3801,7 +3795,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
 
                 #ifdef IMPL_EXECUTE_LOOP
                     term catch_term = term_from_catch_label(mod->module_index, label);
-                    // TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, catch_term);
                 #endif
                 break;
@@ -3814,7 +3807,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 TRACE("catch_end/1, reg=%c%i\n", T_DEST_REG(dreg));
 
                 #ifdef IMPL_EXECUTE_LOOP
-                    // TODO: here just write to y registers is enough
                     WRITE_REGISTER(dreg, term_nil());
                     // C.f. https://www.erlang.org/doc/reference_manual/expressions.html#catch-and-throw
                     switch (term_to_atom_index(x_regs[0])) {
@@ -5556,7 +5548,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: stub, always false
             case OP_IS_BITSTR: {
                 uint32_t label;
                 DECODE_LABEL(label, pc)
@@ -5579,8 +5570,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            // TODO: This opcode is currently uncovered by tests.
-            // We need to implement GC bifs with arity 3, e.g. binary_part/3.
             case OP_GC_BIF3: {
                 uint32_t fail_label;
                 DECODE_LABEL(fail_label, pc);
@@ -5654,7 +5643,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
             }
 
 #if MINIMUM_OTP_COMPILER_VERSION <= 23
-            //TODO: stub, implement recv_mark/1
             //it looks like it can be safely left unimplemented
             case OP_RECV_MARK: {
                 uint32_t label;
@@ -5665,7 +5653,6 @@ HOT_FUNC int scheduler_entry_point(GlobalContext *glb)
                 break;
             }
 
-            //TODO: stub, implement recv_set/1
             //it looks like it can be safely left unimplemented
             case OP_RECV_SET: {
                 uint32_t label;