remove useless if and just return instead...

src/calc_jmp.cpp

@@ -9,16 +9,14 @@ namespace vm
             auto result =
                 std::find_if( vm_entry.begin(), vm_entry.end(), []( const zydis_instr_t &instr_data ) -> bool {
                     // mov/movsx/movzx rax/eax/ax/al, [rsi]
-                    if ( instr_data.instr.operand_count > 1 &&
+                    return ( instr_data.instr.operand_count > 1 &&
                              ( instr_data.instr.mnemonic == ZYDIS_MNEMONIC_MOV ||
                                instr_data.instr.mnemonic == ZYDIS_MNEMONIC_MOVSX ||
                                instr_data.instr.mnemonic == ZYDIS_MNEMONIC_MOVZX ) &&
                              instr_data.instr.operands[ 0 ].type == ZYDIS_OPERAND_TYPE_REGISTER &&
                              util::reg::to64( instr_data.instr.operands[ 0 ].reg.value ) == ZYDIS_REGISTER_RAX &&
                              instr_data.instr.operands[ 1 ].type == ZYDIS_OPERAND_TYPE_MEMORY &&
-                         instr_data.instr.operands[ 1 ].mem.base == ZYDIS_REGISTER_RSI )
-                        return true;
-                    return false;
+                             instr_data.instr.operands[ 1 ].mem.base == ZYDIS_REGISTER_RSI );
                 } );
 
             if ( result == vm_entry.end() )