manchas: Valgrind tiene un bug que nos obliga a modificar el código de...

manchas: Valgrind tiene un bug que nos obliga a modificar el código de Manchas_simple. Integrado a la testsuite de la cátedra.

src/filters/Manchas_simple.asm

@@ -278,7 +278,7 @@ Manchas_simple:
             jnz .x_loop
 
         ; Ya me movi todo un row => decremento los verticales
-        dec r15
+        dec r15d
         ; Si ya llegue abajo del todo salgo, si no repito
         jnz .y_loop
 

src/tests/2_test_diff_cat_asm.py

@@ -21,6 +21,10 @@ for imagen in archivos:
     ok = verificar('Manchas', '100', 5, 'mvec', imagen)
     todos_ok = todos_ok and ok
 
+for imagen in archivos:
+    ok = verificar('Manchas', '100', 5, 'simple', imagen)
+    todos_ok = todos_ok and ok
+
 if todos_ok:
     print(colored("Test de filtros finalizados correctamente", 'green'))
 else:

src/tests/3_correr_test_mem.sh

@@ -55,10 +55,10 @@ for s in ${SIZESMEM[*]}; do
   if [ $ret -ne 0 ]; then exit -1; fi
 done
 
-#for s in ${SIZESMEM[*]}; do
-#  run_test "$TP2ALU" Manchas simple "$TESTINDIR/$img1.$s.bmp" "100"
-#  if [ $ret -ne 0 ]; then exit -1; fi
-#done
+for s in ${SIZESMEM[*]}; do
+  run_test "$TP2ALU" Manchas simple "$TESTINDIR/$img1.$s.bmp" "100"
+  if [ $ret -ne 0 ]; then exit -1; fi
+done
 
 for s in ${SIZESMEM[*]}; do
   run_test "$TP2ALU" Manchas mvec "$TESTINDIR/$img1.$s.bmp" "100"

[Ignacio Esteban Losiggio]

No es un bug de valgrind, es un error muy sutil en la lógica nuestra.