engines/sg: Removing useless variable assignment
authorErwan Velu <erwanaliasr1@gmail.com>
Wed, 8 Sep 2021 20:43:39 +0000 (22:43 +0200)
committerErwan Velu <erwanaliasr1@gmail.com>
Wed, 8 Sep 2021 21:30:48 +0000 (23:30 +0200)
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 <erwanaliasr1@gmail.com>
engines/sg.c

index 592c448890dc5130cf4fbc2c6a145ce57a76a28d..1c0193840df3cacbf276be48542f5cd5af7b80ad 100644 (file)
@@ -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;