VirtualBox

Ignore:
Timestamp:
Aug 16, 2011 10:34:32 AM (13 years ago)
Author:
vboxsync
Message:

VD: Interface cleanup. Merge the two involved structures (generic interface descriptor and callback table) into one, remove the duplicated interface wrappers in the backends and move the interface definitions into separate headers separating public and private interfaces.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Storage/testcase/tstVD.cpp

    r33567 r38469  
    6868    VDGEOMETRY       LCHS = { 0, 0, 0 };
    6969    PVDINTERFACE     pVDIfs = NULL;
    70     VDINTERFACE      VDIError;
    71     VDINTERFACEERROR VDIErrorCallbacks;
     70    VDINTERFACEERROR VDIfError;
    7271
    7372#define CHECK(str) \
    … …  
    8382
    8483    /* Create error interface. */
    85     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    86     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    87     VDIErrorCallbacks.pfnError = tstVDError;
    88     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    89 
    90     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    91                         NULL, &pVDIfs);
     84    VDIfError.pfnError = tstVDError;
     85    VDIfError.pfnMessage = tstVDMessage;
     86
     87    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     88                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    9289    AssertRC(rc);
    9390
    94     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     91    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    9592    CHECK("VDCreate()");
    9693
    … …  
    126123    VDGEOMETRY       LCHS = { 0, 0, 0 };
    127124    PVDINTERFACE     pVDIfs = NULL;
    128     VDINTERFACE      VDIError;
    129     VDINTERFACEERROR VDIErrorCallbacks;
     125    VDINTERFACEERROR VDIfError;
    130126
    131127#define CHECK(str) \
    … …  
    141137
    142138    /* Create error interface. */
    143     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    144     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    145     VDIErrorCallbacks.pfnError = tstVDError;
    146     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    147 
    148     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    149                         NULL, &pVDIfs);
     139    VDIfError.pfnError = tstVDError;
     140    VDIfError.pfnMessage = tstVDMessage;
     141
     142    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     143                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    150144    AssertRC(rc);
    151145
    152     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     146
     147    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    153148    CHECK("VDCreate()");
    154149
    … …  
    511506    uint32_t u32SectorSize = 512;
    512507    PVDINTERFACE     pVDIfs = NULL;
    513     VDINTERFACE      VDIError;
    514     VDINTERFACEERROR VDIErrorCallbacks;
     508    VDINTERFACEERROR VDIfError;
    515509
    516510#define CHECK(str) \
    … …  
    530524
    531525    /* Create error interface. */
    532     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    533     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    534     VDIErrorCallbacks.pfnError = tstVDError;
    535     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    536 
    537     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    538                         NULL, &pVDIfs);
     526    VDIfError.pfnError = tstVDError;
     527    VDIfError.pfnMessage = tstVDMessage;
     528
     529    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     530                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    539531    AssertRC(rc);
    540532
    541533
    542     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     534    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    543535    CHECK("VDCreate()");
    544536
    … …  
    636628    uint32_t u32SectorSize = 512;
    637629    PVDINTERFACE     pVDIfs = NULL;
    638     VDINTERFACE      VDIError;
    639     VDINTERFACEERROR VDIErrorCallbacks;
     630    VDINTERFACEERROR VDIfError;
    640631
    641632#define CHECK(str) \
    … …  
    655646
    656647    /* Create error interface. */
    657     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    658     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    659     VDIErrorCallbacks.pfnError = tstVDError;
    660     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    661 
    662     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    663                         NULL, &pVDIfs);
     648    VDIfError.pfnError = tstVDError;
     649    VDIfError.pfnMessage = tstVDMessage;
     650
     651    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     652                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    664653    AssertRC(rc);
    665654
    666 
    667     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     655    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    668656    CHECK("VDCreate()");
    669657
    … …  
    715703    PVBOXHDD pVD = NULL;
    716704    PVDINTERFACE     pVDIfs = NULL;
    717     VDINTERFACE      VDIError;
    718     VDINTERFACEERROR VDIErrorCallbacks;
     705    VDINTERFACEERROR VDIfError;
    719706
    720707#define CHECK(str) \
    … …  
    730717
    731718    /* Create error interface. */
    732     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    733     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    734     VDIErrorCallbacks.pfnError = tstVDError;
    735     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    736 
    737     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    738                         NULL, &pVDIfs);
     719    VDIfError.pfnError = tstVDError;
     720    VDIfError.pfnMessage = tstVDMessage;
     721
     722    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     723                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    739724    AssertRC(rc);
    740725
    741     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     726    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    742727    CHECK("VDCreate()");
    743728
    … …  
    766751    PVBOXHDD pVD = NULL;
    767752    PVDINTERFACE     pVDIfs = NULL;
    768     VDINTERFACE      VDIError;
    769     VDINTERFACEERROR VDIErrorCallbacks;
     753    VDINTERFACEERROR VDIfError;
    770754
    771755#define CHECK(str) \
    … …  
    781765
    782766    /* Create error interface. */
    783     VDIErrorCallbacks.cbSize = sizeof(VDINTERFACEERROR);
    784     VDIErrorCallbacks.enmInterface = VDINTERFACETYPE_ERROR;
    785     VDIErrorCallbacks.pfnError = tstVDError;
    786     VDIErrorCallbacks.pfnMessage = tstVDMessage;
    787 
    788     rc = VDInterfaceAdd(&VDIError, "tstVD_Error", VDINTERFACETYPE_ERROR, &VDIErrorCallbacks,
    789                         NULL, &pVDIfs);
     767    VDIfError.pfnError = tstVDError;
     768    VDIfError.pfnMessage = tstVDMessage;
     769
     770    rc = VDInterfaceAdd(&VDIfError.Core, "tstVD_Error", VDINTERFACETYPE_ERROR,
     771                        NULL, sizeof(VDINTERFACEERROR), &pVDIfs);
    790772    AssertRC(rc);
    791773
    792     rc = VDCreate(&VDIError, VDTYPE_HDD, &pVD);
     774    rc = VDCreate(pVDIfs, VDTYPE_HDD, &pVD);
    793775    CHECK("VDCreate()");
    794776
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