VirtualBox

Changeset 49940 in vbox for trunk/src


Ignore:
Timestamp:
Dec 16, 2013 6:16:26 PM (11 years ago)
Author:
vboxsync
Message:

VBoxFBOverlay.cpp: Avoid static initializer like the pest, at least if they involve the yet not initialized IPRT! Moved g_VBoxCmdEntriesCache into the VBoxVHWACommandElementProcessor class as a pointer. Also did a s/Assert *(0)/AssertFailed()/g.

Location:
trunk/src/VBox/Frontends/VirtualBox/src
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VirtualBox/src/VBoxFBOverlay.cpp

    r49535 r49940  
    212212};
    213213
    214 static VBoxVHWAEntriesCache g_VBoxCmdEntriesCache;
    215 
    216214static struct VBOXVHWACMD * vhwaHHCmdCreate(VBOXVHWACMD_TYPE type, size_t size)
    217215{
    … …  
    312310    {
    313311        /* @todo: resize */
    314         Assert(0);
     312        AssertFailed();
    315313    }
    316314
    … …  
    334332    }
    335333
    336     Assert(0);
     334    AssertFailed();
    337335    return VBOXVHWA_SURFHANDLE_INVALID;
    338336}
    … …  
    486484    if (!fi.open(QIODevice::ReadOnly))
    487485    {
    488         Assert(0);
     486        AssertFailed();
    489487        return VERR_GENERAL_FAILURE;
    490488    }
    … …  
    920918                    break;
    921919                default:
    922                     Assert(0);
     920                    AssertFailed();
    923921                    break;
    924922            }
    … …  
    958956        return VINF_SUCCESS;
    959957
    960     Assert(0);
     958    AssertFailed();
    961959    VBoxVHWAGlProgram::uninit();
    962960    return VERR_GENERAL_FAILURE;
    … …  
    11071105    if(type & VBOXVHWA_PROGRAM_SRCCOLORKEY)
    11081106    {
    1109         Assert(0);
     1107        AssertFailed();
    11101108        /* disabled for now, not really necessary for video overlaying */
    11111109    }
    … …  
    13601358        break;
    13611359    default:
    1362         Assert(0);
     1360        AssertFailed();
    13631361        break;
    13641362    }
    … …  
    13821380        break;
    13831381    default:
    1384         Assert(0);
     1382        AssertFailed();
    13851383        break;
    13861384    }
    … …  
    15661564    VBOXQGLDBGPRINT(("<?dml?><exec cmd=\"!vbvdbg.ms 0x%p 0n%d 0n%d\">texture info</exec>\n",
    15671565            pvBuf, mRect.width(), mRect.height()));
    1568     Assert(0);
     1566    AssertFailed();
    15691567
    15701568    free(pvBuf);
    … …  
    22052203    if(!(pCmd->SurfInfo.flags & VBOXVHWA_SD_CAPS))
    22062204    {
    2207         Assert(0);
     2205        AssertFailed();
    22082206        pCmd->u.out.ErrInfo = -1;
    22092207        return VINF_SUCCESS;
    … …  
    22132211    {
    22142212#ifdef DEBUGVHWASTRICT
    2215         Assert(0);
     2213        AssertFailed();
    22162214#endif
    22172215        pCmd->u.out.ErrInfo = -1;
    … …  
    22252223        {
    22262224#ifdef DEBUG_misha
    2227             Assert(0);
     2225            AssertFailed();
    22282226#endif
    22292227            pCmd->u.out.ErrInfo = -1;
    … …  
    22402238    {
    22412239#ifdef DEBUGVHWASTRICT
    2242         Assert (0);
     2240        AssertFailed();
    22432241#endif
    22442242        pCmd->u.out.ErrInfo = -1;
    … …  
    22512249        if (!(pCmd->SurfInfo.flags & VBOXVHWA_SD_PIXELFORMAT))
    22522250        {
    2253             Assert (0);
     2251            AssertFailed();
    22542252            pCmd->u.out.ErrInfo = -1;
    22552253            return VINF_SUCCESS;
    … …  
    22612259                    || pCmd->SurfInfo.PixelFormat.c.rgbBitCount != 24)
    22622260            {
    2263                 Assert (0);
     2261                AssertFailed();
    22642262                pCmd->u.out.ErrInfo = -1;
    22652263                return VINF_SUCCESS;
    … …  
    22852283        else
    22862284        {
    2287             Assert (0);
     2285            AssertFailed();
    22882286            pCmd->u.out.ErrInfo = -1;
    22892287            return VINF_SUCCESS;
    … …  
    23052303        if(mSurfHandleTable.get(handle))
    23062304        {
    2307             Assert(0);
     2305            AssertFailed();
    23082306            return VERR_GENERAL_FAILURE;
    23092307        }
    … …  
    32313229        else
    32323230        {
    3233             Assert(0);
     3231            AssertFailed();
    32343232        }
    32353233
    … …  
    37913789            case 24:
    37923790#ifdef DEBUG_misha
    3793                 Assert(0);
     3791                AssertFailed();
    37943792#endif
    37953793                break;
    37963794            case 8:
    37973795#ifdef DEBUG_misha
    3798                 Assert(0);
     3796                AssertFailed();
    37993797#endif
    38003798                g = b = 0;
    … …  
    38033801            case 1:
    38043802#ifdef DEBUG_misha
    3805                 Assert(0);
     3803                AssertFailed();
    38063804#endif
    38073805                r = 1;
    … …  
    38113809            default:
    38123810#ifdef DEBUG_misha
    3813                 Assert(0);
     3811                AssertFailed();
    38143812#endif
    38153813                remind = true;
    … …  
    40274025            break;
    40284026        default:
    4029             Assert(0);
     4027            AssertFailed();
    40304028            mBitsPerPixel = 0;
    40314029            mBitsPerPixelTex = 0;
    … …  
    40524050        case 24:
    40534051#ifdef DEBUG_misha
    4054             Assert(0);
     4052            AssertFailed();
    40554053#endif
    40564054            mInternalFormat = 3;//GL_RGB;
    … …  
    40634061        case 16:
    40644062#ifdef DEBUG_misha
    4065             Assert(0);
     4063            AssertFailed();
    40664064#endif
    40674065            mInternalFormat = GL_RGB5;
    … …  
    40744072        case 8:
    40754073#ifdef DEBUG_misha
    4076             Assert(0);
     4074            AssertFailed();
    40774075#endif
    40784076            mInternalFormat = 1;//GL_RGB;
    … …  
    40834081        case 1:
    40844082#ifdef DEBUG_misha
    4085             Assert(0);
     4083            AssertFailed();
    40864084#endif
    40874085            mInternalFormat = 1;
    … …  
    40924090        default:
    40934091#ifdef DEBUG_misha
    4094             Assert(0);
     4092            AssertFailed();
    40954093#endif
    40964094            mBitsPerPixel = 0;
    … …  
    48244822#endif
    48254823        default:
    4826             Assert(0);
     4824            AssertFailed();
    48274825            pCmd->rc = VERR_NOT_IMPLEMENTED;
    48284826            break;
    … …  
    49454943#endif
    49464944        default:
    4947             Assert(0);
     4945            AssertFailed();
    49484946    }
    49494947}
    … …  
    49594957
    49604958    RTListInit(&mCommandList);
     4959
     4960    m_pCmdEntryCache = new VBoxVHWAEntriesCache;
    49614961}
    49624962
    … …  
    49674967
    49684968    RTCritSectDelete(&mCritSect);
     4969
     4970    delete m_pCmdEntryCache;
    49694971}
    49704972
    … …  
    49784980    RTCritSectEnter(&mCritSect);
    49794981
    4980     VBoxVHWACommandElement * pCmd = g_VBoxCmdEntriesCache.alloc();
     4982    VBoxVHWACommandElement * pCmd = m_pCmdEntryCache->alloc();
    49814983    if(!pCmd)
    49824984    {
    … …  
    50795081
    50805082    if (pEl)
    5081         g_VBoxCmdEntriesCache.free(pEl);
     5083        m_pCmdEntryCache->free(pEl);
    50825084}
    50835085
    … …  
    51735175        case VBOXVHWA_PIPECMD_FUNC:
    51745176            /* should not happen, don't handle this for now */
    5175             Assert(0);
     5177            AssertFailed();
    51765178            break;
    51775179#endif
    … …  
    51805182        default:
    51815183            /* should not happen, don't handle this for now */
    5182             Assert(0);
     5184            AssertFailed();
    51835185            break;
    51845186        }
    51855187
    51865188        RTListNodeRemove(&pCur->ListNode);
    5187         g_VBoxCmdEntriesCache.free(pCur);
     5189        m_pCmdEntryCache->free(pCur);
    51885190    }
    51895191
    … …  
    52605262                        }
    52615263                        default:
    5262                             Assert(0);
     5264                            AssertFailed();
    52635265                            break;
    52645266                    }
    … …  
    53065308            }
    53075309            default:
    5308                 Assert(0);
     5310                AssertFailed();
    53095311                break;
    53105312        }

  • trunk/src/VBox/Frontends/VirtualBox/src/VBoxFBOverlay.h

    r49535 r49940  
    13071307};
    13081308
     1309class VBoxVHWAEntriesCache;
    13091310class VBoxVHWACommandElementProcessor
    13101311{
    … …  
    13311332    bool mbResetting;
    13321333    uint32_t mcDisabled;
     1334    VBoxVHWAEntriesCache *m_pCmdEntryCache;
    13331335};
    13341336
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