VirtualBox

Ignore:
Timestamp:
Dec 22, 2018 1:43:57 AM (6 years ago)
Author:
vboxsync
Message:

UIGuestControlInterface.cpp: Consistent switch style, no else after return, include err.h. bugref:9344

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VirtualBox/src/guestctrl/UIGuestControlInterface.cpp

    r75425 r76361  
    3333
    3434/* Misc. includes: */
     35# include <iprt/err.h>
    3536# include <iprt/getopt.h>
    3637
    … …  
    4546#define GCTLCMD_COMMON_OPT_SESSION_ID       994 /**< The --sessionid option number. */
    4647
    47 #define RETURN_ERROR(strError)     \
    48     {                              \
    49     m_strStatus.append(strError);  \
    50     return false;                  \
    51     }
    52 
    53 #define RETURN_MESSAGE(strMessage)   \
    54     {                                \
    55     m_strStatus.append(strMessage);  \
    56     return true;                     \
    57     }
     48#define RETURN_ERROR(strError)         \
     49    do {                               \
     50        m_strStatus.append(strError);  \
     51        return false;                  \
     52    } while (0)
     53
     54#define RETURN_MESSAGE(strMessage)       \
     55    do {                                 \
     56        m_strStatus.append(strMessage);  \
     57        return true;                     \
     58    } while (0)
    5859
    5960#define GCTLCMD_COMMON_OPTION_DEFS() \
    … …  
    7879    switch(type)
    7980    {
    80         case (KFsObjType_Unknown):
     81        case KFsObjType_Unknown:
    8182            strType = "Unknown";
    8283            break;
    83     case (KFsObjType_Fifo):
    84         strType = "Fifo";
    85         break;
    86 
    87     case (KFsObjType_DevChar):
    88         strType = "DevChar";
    89         break;
    90 
    91     case (KFsObjType_Directory):
    92         strType = "Directory";
    93         break;
    94 
    95     case (KFsObjType_DevBlock):
    96         strType = "DevBlock";
    97         break;
    98 
    99     case (KFsObjType_File):
    100         strType = "File";
    101         break;
    102     case (KFsObjType_Symlink):
    103         strType = "Symlink";
    104         break;
    105     case (KFsObjType_Socket):
    106         strType = "Socket";
    107         break;
    108 
    109     case (KFsObjType_WhiteOut):
    110         strType = "WhiteOut";
    111         break;
    112     default:
    113         strType = "Unknown";
    114         break;
     84        case KFsObjType_Fifo:
     85            strType = "Fifo";
     86            break;
     87        case KFsObjType_DevChar:
     88            strType = "DevChar";
     89            break;
     90        case KFsObjType_Directory:
     91            strType = "Directory";
     92            break;
     93        case KFsObjType_DevBlock:
     94            strType = "DevBlock";
     95            break;
     96        case KFsObjType_File:
     97            strType = "File";
     98            break;
     99        case KFsObjType_Symlink:
     100            strType = "Symlink";
     101            break;
     102        case KFsObjType_Socket:
     103            strType = "Socket";
     104            break;
     105        case KFsObjType_WhiteOut:
     106            strType = "WhiteOut";
     107            break;
     108        default:
     109            strType = "Unknown";
     110            break;
    115111    }
    116112    return strType;
    … …  
    130126    switch (getOptErrorCode)
    131127    {
    132         case (VERR_GETOPT_UNKNOWN_OPTION):
     128        case VERR_GETOPT_UNKNOWN_OPTION:
    133129            errorString = errorString.append("RTGetOpt: Command line option not recognized.");
    134130            break;
    135         case (VERR_GETOPT_REQUIRED_ARGUMENT_MISSING):
     131        case VERR_GETOPT_REQUIRED_ARGUMENT_MISSING:
    136132            errorString = errorString.append("RTGetOpt: Command line option needs argument.");
    137133            break;
    138         case (VERR_GETOPT_INVALID_ARGUMENT_FORMAT):
     134        case VERR_GETOPT_INVALID_ARGUMENT_FORMAT:
    139135            errorString = errorString.append("RTGetOpt: Command line option has argument with bad format.");
    140136            break;
    141         case (VINF_GETOPT_NOT_OPTION):
     137        case VINF_GETOPT_NOT_OPTION:
    142138            errorString = errorString.append("RTGetOpt: Not an option.");
    143139            break;
    144         case (VERR_GETOPT_INDEX_MISSING):
     140        case VERR_GETOPT_INDEX_MISSING:
    145141            errorString = errorString.append("RTGetOpt: Command line option needs an index.");
    146142            break;
    … …  
    238234                /* Allow only a single NOT_OPTION */
    239235                else
    240                     RETURN_ERROR(generateErrorString(ch, ValueUnion))
     236                    RETURN_ERROR(generateErrorString(ch, ValueUnion));
    241237
    242238                break;
    243239            default:
    244                 RETURN_ERROR(generateErrorString(ch, ValueUnion))
     240                RETURN_ERROR(generateErrorString(ch, ValueUnion));
    245241        }
    246242    }
    … …  
    307303                /* Allow only a single NOT_OPTION */
    308304                else
    309                     RETURN_ERROR(generateErrorString(ch, ValueUnion))
     305                    RETURN_ERROR(generateErrorString(ch, ValueUnion));
    310306
    311307                break;
    312308            default:
    313                 RETURN_ERROR(generateErrorString(ch, ValueUnion))
     309                RETURN_ERROR(generateErrorString(ch, ValueUnion));
    314310        }
    315311    }
    … …  
    331327
    332328    if (!isADirectory && !isAFile)
    333         RETURN_ERROR("Specified object does not exist")
     329        RETURN_ERROR("Specified object does not exist");
    334330
    335331    CGuestFsObjInfo fsObjectInfo = guestSession.FsObjQueryInfo(commandData.m_strPath, false /*BOOL aFollowSymlinks*/);
    … …  
    410406                break;
    411407            default:
    412                 RETURN_ERROR(generateErrorString(ch, ValueUnion))
     408                RETURN_ERROR(generateErrorString(ch, ValueUnion));
    413409        }
    414410    }
    415411    if (commandData.m_strExePath.isEmpty())
    416         RETURN_ERROR(QString(m_strHelp).append("Syntax error! No executable is given\n"))
     412        RETURN_ERROR(QString(m_strHelp).append("Syntax error! No executable is given\n"));
    417413
    418414    CGuestSession guestSession;
    … …  
    426422{
    427423    if (commandData.m_bSessionNameGiven && commandData.m_strSessionName.isEmpty())
    428         RETURN_ERROR(QString(m_strHelp).append("'Session Name' is not name valid\n"))
     424        RETURN_ERROR(QString(m_strHelp).append("'Session Name' is not name valid\n"));
    429425
    430426    /* Check if sessionname and sessionid are both supplied */
    431427    if (commandData.m_bSessionIdGiven && commandData.m_bSessionNameGiven)
    432         RETURN_ERROR(QString(m_strHelp).append("Both 'Session Name' and 'Session Id' are supplied\n"))
     428        RETURN_ERROR(QString(m_strHelp).append("Both 'Session Name' and 'Session Id' are supplied\n"));
    433429
    434430    /* If sessionid is given then look for the session. if not found return without starting the process: */
    … …  
    437433        if (!findSession(commandData.m_uSessionId, outGuestSession))
    438434        {
    439             RETURN_ERROR(QString(m_strHelp).append("No session with id %1 found.\n").arg(commandData.m_uSessionId))
     435            RETURN_ERROR(QString(m_strHelp).append("No session with id %1 found.\n").arg(commandData.m_uSessionId));
    440436        }
    441437    }
    … …  
    453449        return true;
    454450    /* if neither sessionname and session id is given then create a new session */
    455     else
    456     {
    457         if (!createSession(commandData, outGuestSession))
    458             return false;
    459     }
     451    if (!createSession(commandData, outGuestSession))
     452        return false;
    460453    return true;
    461454}
    … …  
    507500                if (commandData.m_strSessionName.isEmpty())
    508501                {
    509                     RETURN_ERROR(QString("'Session Name' is not name valid\n").append(m_strHelp))
     502                    RETURN_ERROR(QString("'Session Name' is not name valid\n").append(m_strHelp));
    510503                }
    511504                break;
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette