engines/sg: Removing useless variable assignment

ret is set to -1 but the break statement will not use this value. So let's remove this useless assignment which could be confusing. Signed-off-by: Erwan Velu <[email protected]>
axboe · Sep 8, 2021 · 691df59 · 691df59
1 parent 824d818
commit 691df59
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/engines/sg.c b/engines/sg.c
@@ -471,10 +471,9 @@ static enum fio_q_status fio_sgio_rw_doio(struct thread_data *td,
 			if (__io_u == io_u)
 				break;
 
-			if (io_u_sync_complete(td, __io_u)) {
-				ret = -1;
+			if (io_u_sync_complete(td, __io_u))
 				break;
-			}
+
 		} while (1);
 
 		return FIO_Q_COMPLETED;