https://bugs.documentfoundation.org/show_bug.cgi?id=147132

Hossein <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Whiteboard|target:7.4.0 target:7.5.0   |target:7.4.0 target:7.5.0
                   |                            |reviewed:2022

--- Comment #8 from Hossein <[email protected]> ---
Re-evaluating the EasyHack in 2022

This enhancement is still relevant. One may search for this statement to find
instances of the proposed change in basic/source/runtime/methods.cxx:

    StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT );

The result of this search should be checked to makes sure that there is no
other expression after the matching else.

There are somehow similar conditions:

    if ( ... )
    {
        StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT );
        return;
    }

which can also be converted to a shorter form:

    if ( ... )
        return StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT );

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to