VirtualBox

Changeset 107741 in vbox


Ignore:
Timestamp:
Jan 14, 2025 9:48:26 AM (2 months ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
166848
Message:

src/VBox/Frontends/VBoxManage/VBoxManageInfo.cpp: Fixed warnings found by Parfait (unused assignment). jiraref:VBP-1424

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageInfo.cpp

    r107267 r107741  
    11491149            {
    11501150                Bstr settingsFilePath;
    1151                 hrc = machine->COMGETTER(SettingsFilePath)(settingsFilePath.asOutParam());
    1152                 RTPrintf(Info::tr("Config file:     %ls\n"), settingsFilePath.raw());
     1151                CHECK_ERROR(machine, COMGETTER(SettingsFilePath)(settingsFilePath.asOutParam()));
     1152                RTPrintf(Info::tr("Config file:     %ls\n"), SUCCEEDED(hrc) ? settingsFilePath.raw() : L"<Error>");
    11531153
    11541154                Bstr strCipher;
    … …  
    11651165
    11661166                ComPtr<IVirtualBoxErrorInfo> accessError;
    1167                 hrc = machine->COMGETTER(AccessError)(accessError.asOutParam());
    1168                 RTPrintf(Info::tr("Access error details:\n"));
    1169                 ErrorInfo ei(accessError);
    1170                 GluePrintErrorInfo(ei);
    1171                 RTPrintf("\n");
     1167                CHECK_ERROR(machine, COMGETTER(AccessError)(accessError.asOutParam()));
     1168                if (SUCCEEDED(hrc))
     1169                {
     1170                    RTPrintf(Info::tr("Access error details:\n"));
     1171                    ErrorInfo ei(accessError);
     1172                    GluePrintErrorInfo(ei);
     1173                    RTPrintf("\n");
     1174                }
    11721175            }
    11731176        }
    … …  
    22252228    if (SUCCEEDED(hrc))
    22262229    {
    2227         const char *pszDrv   = Info::tr("Unknown");
    2228         const char *pszCtrl  = Info::tr("Unknown");
    2229         const char *pszCodec = Info::tr("Unknown");
     2230        const char *pszDrv   = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
     2231        const char *pszCtrl  = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
     2232        const char *pszCodec = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
    22302233        BOOL fEnabled;
    22312234        hrc = audioAdapter->COMGETTER(Enabled)(&fEnabled);
    … …  
    22342237            AudioDriverType_T enmDrvType;
    22352238            hrc = audioAdapter->COMGETTER(AudioDriver)(&enmDrvType);
    2236             switch (enmDrvType)
    2237             {
    2238                 case AudioDriverType_Default:
    2239                     if (details == VMINFO_MACHINEREADABLE)
    2240                         pszDrv = "default";
    2241                     else
    2242                         pszDrv = Info::tr("Default");
    2243                     break;
    2244                 case AudioDriverType_Null:
    2245                     if (details == VMINFO_MACHINEREADABLE)
    2246                         pszDrv = "null";
    2247                     else
    2248                         pszDrv = Info::tr("Null");
    2249                     break;
    2250                 case AudioDriverType_OSS:
    2251                     if (details == VMINFO_MACHINEREADABLE)
    2252                         pszDrv = "oss";
    2253                     else
    2254                         pszDrv = "OSS";
    2255                     break;
    2256                 case AudioDriverType_ALSA:
    2257                     if (details == VMINFO_MACHINEREADABLE)
    2258                         pszDrv = "alsa";
    2259                     else
    2260                         pszDrv = "ALSA";
    2261                     break;
    2262                 case AudioDriverType_Pulse:
    2263                     if (details == VMINFO_MACHINEREADABLE)
    2264                         pszDrv = "pulse";
    2265                     else
    2266                         pszDrv = "PulseAudio";
    2267                     break;
    2268                 case AudioDriverType_WinMM:
    2269                     if (details == VMINFO_MACHINEREADABLE)
    2270                         pszDrv = "winmm";
    2271                     else
    2272                         pszDrv = "WINMM";
    2273                     break;
    2274                 case AudioDriverType_DirectSound:
    2275                     if (details == VMINFO_MACHINEREADABLE)
    2276                         pszDrv = "dsound";
    2277                     else
    2278                         pszDrv = "DirectSound";
    2279                     break;
    2280                 case AudioDriverType_WAS:
    2281                     if (details == VMINFO_MACHINEREADABLE)
    2282                         pszDrv = "was";
    2283                     else
    2284                         pszDrv = "Windows Audio Session (WAS)";
    2285                     break;
    2286                 case AudioDriverType_CoreAudio:
    2287                     if (details == VMINFO_MACHINEREADABLE)
    2288                         pszDrv = "coreaudio";
    2289                     else
    2290                         pszDrv = "CoreAudio";
    2291                     break;
    2292                 case AudioDriverType_SolAudio:
    2293                     if (details == VMINFO_MACHINEREADABLE)
    2294                         pszDrv = "solaudio";
    2295                     else
    2296                         pszDrv = "SolAudio";
    2297                     break;
    2298                 default:
    2299                     if (details == VMINFO_MACHINEREADABLE)
    2300                         pszDrv = "unknown";
    2301                     break;
    2302             }
     2239            if (SUCCEEDED(hrc))
     2240            {
     2241                switch (enmDrvType)
     2242                {
     2243                    case AudioDriverType_Default:
     2244                        if (details == VMINFO_MACHINEREADABLE)
     2245                            pszDrv = "default";
     2246                        else
     2247                            pszDrv = Info::tr("Default");
     2248                        break;
     2249                    case AudioDriverType_Null:
     2250                        if (details == VMINFO_MACHINEREADABLE)
     2251                            pszDrv = "null";
     2252                        else
     2253                            pszDrv = Info::tr("Null");
     2254                        break;
     2255                    case AudioDriverType_OSS:
     2256                        if (details == VMINFO_MACHINEREADABLE)
     2257                            pszDrv = "oss";
     2258                        else
     2259                            pszDrv = "OSS";
     2260                        break;
     2261                    case AudioDriverType_ALSA:
     2262                        if (details == VMINFO_MACHINEREADABLE)
     2263                            pszDrv = "alsa";
     2264                        else
     2265                            pszDrv = "ALSA";
     2266                        break;
     2267                    case AudioDriverType_Pulse:
     2268                        if (details == VMINFO_MACHINEREADABLE)
     2269                            pszDrv = "pulse";
     2270                        else
     2271                            pszDrv = "PulseAudio";
     2272                        break;
     2273                    case AudioDriverType_WinMM:
     2274                        if (details == VMINFO_MACHINEREADABLE)
     2275                            pszDrv = "winmm";
     2276                        else
     2277                            pszDrv = "WINMM";
     2278                        break;
     2279                    case AudioDriverType_DirectSound:
     2280                        if (details == VMINFO_MACHINEREADABLE)
     2281                            pszDrv = "dsound";
     2282                        else
     2283                            pszDrv = "DirectSound";
     2284                        break;
     2285                    case AudioDriverType_WAS:
     2286                        if (details == VMINFO_MACHINEREADABLE)
     2287                            pszDrv = "was";
     2288                        else
     2289                            pszDrv = "Windows Audio Session (WAS)";
     2290                        break;
     2291                    case AudioDriverType_CoreAudio:
     2292                        if (details == VMINFO_MACHINEREADABLE)
     2293                            pszDrv = "coreaudio";
     2294                        else
     2295                            pszDrv = "CoreAudio";
     2296                        break;
     2297                    case AudioDriverType_SolAudio:
     2298                        if (details == VMINFO_MACHINEREADABLE)
     2299                            pszDrv = "solaudio";
     2300                        else
     2301                            pszDrv = "SolAudio";
     2302                        break;
     2303                    default:
     2304                        if (details == VMINFO_MACHINEREADABLE)
     2305                            pszDrv = "unknown";
     2306                        break;
     2307                }
     2308            }
     2309
    23032310            AudioControllerType_T enmCtrlType;
    23042311            hrc = audioAdapter->COMGETTER(AudioController)(&enmCtrlType);
    2305             switch (enmCtrlType)
    2306             {
    2307                 case AudioControllerType_AC97:
    2308                     if (details == VMINFO_MACHINEREADABLE)
    2309                         pszCtrl = "ac97";
    2310                     else
    2311                         pszCtrl = "AC97";
    2312                     break;
    2313                 case AudioControllerType_SB16:
    2314                     if (details == VMINFO_MACHINEREADABLE)
    2315                         pszCtrl = "sb16";
    2316                     else
    2317                         pszCtrl = "SB16";
    2318                     break;
    2319                 case AudioControllerType_HDA:
    2320                     if (details == VMINFO_MACHINEREADABLE)
    2321                         pszCtrl = "hda";
    2322                     else
    2323                         pszCtrl = "HDA";
    2324                     break;
    2325                 default:
    2326                     break;
    2327             }
     2312            if (SUCCEEDED(hrc))
     2313            {
     2314                switch (enmCtrlType)
     2315                {
     2316                    case AudioControllerType_AC97:
     2317                        if (details == VMINFO_MACHINEREADABLE)
     2318                            pszCtrl = "ac97";
     2319                        else
     2320                            pszCtrl = "AC97";
     2321                        break;
     2322                    case AudioControllerType_SB16:
     2323                        if (details == VMINFO_MACHINEREADABLE)
     2324                            pszCtrl = "sb16";
     2325                        else
     2326                            pszCtrl = "SB16";
     2327                        break;
     2328                    case AudioControllerType_HDA:
     2329                        if (details == VMINFO_MACHINEREADABLE)
     2330                            pszCtrl = "hda";
     2331                        else
     2332                            pszCtrl = "HDA";
     2333                        break;
     2334                    default:
     2335                        break;
     2336                }
     2337            }
     2338
    23282339            AudioCodecType_T enmCodecType;
    23292340            hrc = audioAdapter->COMGETTER(AudioCodec)(&enmCodecType);
    2330             switch (enmCodecType)
    2331             {
    2332                 case AudioCodecType_SB16:
    2333                     pszCodec = "SB16";
    2334                     break;
    2335                 case AudioCodecType_STAC9700:
    2336                     pszCodec = "STAC9700";
    2337                     break;
    2338                 case AudioCodecType_AD1980:
    2339                     pszCodec = "AD1980";
    2340                     break;
    2341                 case AudioCodecType_STAC9221:
    2342                     pszCodec = "STAC9221";
    2343                     break;
    2344                 case AudioCodecType_Null: break; /* Shut up MSC. */
    2345                 default:                  break;
     2341            if (SUCCEEDED(hrc))
     2342            {
     2343                switch (enmCodecType)
     2344                {
     2345                    case AudioCodecType_SB16:
     2346                        pszCodec = "SB16";
     2347                        break;
     2348                    case AudioCodecType_STAC9700:
     2349                        pszCodec = "STAC9700";
     2350                        break;
     2351                    case AudioCodecType_AD1980:
     2352                        pszCodec = "AD1980";
     2353                        break;
     2354                    case AudioCodecType_STAC9221:
     2355                        pszCodec = "STAC9221";
     2356                        break;
     2357                    case AudioCodecType_Null: break; /* Shut up MSC. */
     2358                    default:                  break;
     2359                }
    23462360            }
    23472361        }
    … …  
    23662380    /* Shared clipboard */
    23672381    {
    2368         const char *psz;
    2369         ClipboardMode_T enmMode = (ClipboardMode_T)0;
     2382        const char *psz = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
     2383        ClipboardMode_T enmMode;
    23702384        hrc = machine->COMGETTER(ClipboardMode)(&enmMode);
    2371         switch (enmMode)
    2372         {
    2373             case ClipboardMode_Disabled:
    2374                 psz = "disabled";
    2375                 break;
    2376             case ClipboardMode_HostToGuest:
    2377                 psz = details == VMINFO_MACHINEREADABLE ? "hosttoguest" : Info::tr("HostToGuest");
    2378                 break;
    2379             case ClipboardMode_GuestToHost:
    2380                 psz = details == VMINFO_MACHINEREADABLE ? "guesttohost" : Info::tr("GuestToHost");
    2381                 break;
    2382             case ClipboardMode_Bidirectional:
    2383                 psz = details == VMINFO_MACHINEREADABLE ? "bidirectional" : Info::tr("Bidirectional");
    2384                 break;
    2385             default:
    2386                 psz = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
    2387                 break;
     2385        if (SUCCEEDED(hrc))
     2386        {
     2387            switch (enmMode)
     2388            {
     2389                case ClipboardMode_Disabled:
     2390                    psz = "disabled";
     2391                    break;
     2392                case ClipboardMode_HostToGuest:
     2393                    psz = details == VMINFO_MACHINEREADABLE ? "hosttoguest" : Info::tr("HostToGuest");
     2394                    break;
     2395                case ClipboardMode_GuestToHost:
     2396                    psz = details == VMINFO_MACHINEREADABLE ? "guesttohost" : Info::tr("GuestToHost");
     2397                    break;
     2398                case ClipboardMode_Bidirectional:
     2399                    psz = details == VMINFO_MACHINEREADABLE ? "bidirectional" : Info::tr("Bidirectional");
     2400                    break;
     2401                default:
     2402                    break;
     2403            }
    23882404        }
    23892405        SHOW_UTF8_STRING("clipboard", Info::tr("Clipboard Mode:"), psz);
    … …  
    23952411    /* Drag and drop */
    23962412    {
    2397         const char *psz;
     2413        const char *psz = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
    23982414        DnDMode_T enmMode;
    23992415        hrc = machine->COMGETTER(DnDMode)(&enmMode);
    2400         switch (enmMode)
    2401         {
    2402             case DnDMode_Disabled:
    2403                 psz = "disabled";
    2404                 break;
    2405             case DnDMode_HostToGuest:
    2406                 psz = details == VMINFO_MACHINEREADABLE ? "hosttoguest" : Info::tr("HostToGuest");
    2407                 break;
    2408             case DnDMode_GuestToHost:
    2409                 psz = details == VMINFO_MACHINEREADABLE ? "guesttohost" : Info::tr("GuestToHost");
    2410                 break;
    2411             case DnDMode_Bidirectional:
    2412                 psz = details == VMINFO_MACHINEREADABLE ? "bidirectional" : Info::tr("Bidirectional");
    2413                 break;
    2414             default:
    2415                 psz = details == VMINFO_MACHINEREADABLE ? "unknown" : Info::tr("Unknown");
    2416                 break;
     2416        if (SUCCEEDED(hrc))
     2417        {
     2418            switch (enmMode)
     2419            {
     2420                case DnDMode_Disabled:
     2421                    psz = "disabled";
     2422                    break;
     2423                case DnDMode_HostToGuest:
     2424                    psz = details == VMINFO_MACHINEREADABLE ? "hosttoguest" : Info::tr("HostToGuest");
     2425                    break;
     2426                case DnDMode_GuestToHost:
     2427                    psz = details == VMINFO_MACHINEREADABLE ? "guesttohost" : Info::tr("GuestToHost");
     2428                    break;
     2429                case DnDMode_Bidirectional:
     2430                    psz = details == VMINFO_MACHINEREADABLE ? "bidirectional" : Info::tr("Bidirectional");
     2431                    break;
     2432                default:
     2433                    break;
     2434            }
    24172435        }
    24182436        SHOW_UTF8_STRING("draganddrop", Info::tr("Drag and drop Mode:"), psz);
    … …  
    27582776        ComPtr<IBandwidthControl> bwCtrl;
    27592777        CHECK_ERROR_RET(machine, COMGETTER(BandwidthControl)(bwCtrl.asOutParam()), hrc);
    2760 
    2761         hrc = showBandwidthGroups(bwCtrl, details);
    2762     }
    2763 
     2778        showBandwidthGroups(bwCtrl, details);
     2779    }
    27642780
    27652781    /*
    … …  
    29072923        {
    29082924            hrc = progress->COMGETTER(Completed)(&fStarted);
    2909             fStarted = !fStarted;
     2925            if (SUCCEEDED(hrc))
     2926                fStarted = !fStarted;
    29102927        }
    29112928        SHOW_BOOL_VALUE_EX("recording_started", Info::tr("Recording status:"), fStarted, Info::tr("started"), Info::tr("stopped"));
    … …  
    32903307
    32913308        /* open an existing session for the VM */
    3292         hrc = machine->LockMachine(a->session, LockType_Shared);
     3309        CHECK_ERROR(machine, LockMachine(a->session, LockType_Shared));
    32933310        if (SUCCEEDED(hrc))
     3311        {
    32943312            /* get the session machine */
    3295             hrc = a->session->COMGETTER(Machine)(machine.asOutParam());
    3296 
    3297         hrc = showVMInfo(a->virtualBox, machine, a->session, details);
    3298 
    3299         a->session->UnlockMachine();
     3313            CHECK_ERROR(a->session, COMGETTER(Machine)(machine.asOutParam()));
     3314            if (SUCCEEDED(hrc))
     3315                hrc = showVMInfo(a->virtualBox, machine, a->session, details);
     3316
     3317            CHECK_ERROR(a->session, UnlockMachine());
     3318        }
    33003319    }
    33013320
Note: See TracChangeset for help on using the changeset viewer.

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