Hurrison

if aux is set to tuple() in the line above, aux[-1] will throw exception. `aux[-1] if aux else 0` will ensure aux is not None and its length is not...

This line attempts to assign the character following a backslash (\) to tmp->ch, but it should assign the current character pointed by regexp after increment, not the next one

Missing 'break' statement at the end of the case block, causing fall-through to the next case.

This condition checks the length of 'param2' (password) instead of 'param3' (ent_username) before adding '-u' and 'ent_username' to the arguments. It should check if 'param3' is not empty. This condition...

The variable 'wisdomFilePath' is already a complete path constructed with 'config_directory'. Joining it again with 'current_directory' could lead to an incorrect path.

`Gui.cmd.get_command_parts()` is inside a loop but calls a method that likely returns the same result in each iteration