Changeset 30217 in vbox for trunk/src

Timestamp:

Jun 16, 2010 1:43:23 AM (15 years ago)

Author:

vboxsync

Message:

VMM: VCC100 warnings.

Location:

trunk/src/VBox/VMM

Files:

• PDMDriver.cpp (modified) (1 diff)
• testcase/tstHelp.h (modified) (3 diffs)
• testcase/tstVMStructSize.cpp (modified) (7 diffs)

trunk/src/VBox/VMM/PDMDriver.cpp

@@ -109 +109 @@
 
     AssertRelease(!(RT_OFFSETOF(PDMDRVINS, achInstanceData) & 15));
-    PPDMDRVINS pDrvInsAssert;
+    PPDMDRVINS pDrvInsAssert; NOREF(pDrvInsAssert);
+    AssertCompile(sizeof(pDrvInsAssert->Internal.s) <= sizeof(pDrvInsAssert->Internal.padding));
     AssertRelease(sizeof(pDrvInsAssert->Internal.s) <= sizeof(pDrvInsAssert->Internal.padding));
 

trunk/src/VBox/VMM/testcase/tstHelp.h

@@ -103 +103 @@
     do \
     { \
-        strct *p; \
+        strct *p = NULL; NOREF(p); \
         if (sizeof(p->member.s) > sizeof(p->member.padding)) \
         { \
@@ -124 +124 @@
     do \
     { \
-        strct *p; \
+        strct *p = NULL; NOREF(p); \
         if (sizeof(p->s) > sizeof(p->padding)) \
         { \
@@ -139 +139 @@
     do \
     { \
-        strct *p; \
+        strct *p = NULL; NOREF(p); \
         if (sizeof(p->member) > sizeof(p->pad_member)) \
         { \

trunk/src/VBox/VMM/testcase/tstVMStructSize.cpp

@@ -75 +75 @@
         CHECK_PADDING(VM, member, align); \
         CHECK_MEMBER_ALIGNMENT(VM, member, align); \
-        VM *p; \
+        VM *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: VM::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -89 +89 @@
         CHECK_PADDING(VMCPU, member, align); \
         CHECK_MEMBER_ALIGNMENT(VMCPU, member, align); \
-        VMCPU *p; \
+        VMCPU *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: VMCPU::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -111 +111 @@
         CHECK_PADDING(UVM, member, align); \
         CHECK_MEMBER_ALIGNMENT(UVM, member, align); \
-        UVM *p; \
+        UVM *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: UVM::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -124 +124 @@
         CHECK_PADDING(UVMCPU, member, align); \
         CHECK_MEMBER_ALIGNMENT(UVMCPU, member, align); \
-        UVMCPU *p; \
+        UVMCPU *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: UVMCPU::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -137 +137 @@
         CHECK_PADDING(GVM, member, align); \
         CHECK_MEMBER_ALIGNMENT(GVM, member, align); \
-        GVM *p; \
+        GVM *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: GVM::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -150 +150 @@
         CHECK_PADDING(GVMCPU, member, align); \
         CHECK_MEMBER_ALIGNMENT(GVMCPU, member, align); \
-        GVMCPU *p; \
+        GVMCPU *p = NULL; NOREF(p); \
         if (sizeof(p->member.padding) >= (ssize_t)sizeof(p->member.s) + 128 + sizeof(p->member.s) / 20) \
             printf("warning: GVMCPU::%-8s: padding=%-5d s=%-5d -> %-4d  suggest=%-5u\n", \
@@ -235 +235 @@
     CHECK_MEMBER_ALIGNMENT(VM, selm.s.Tss, 16);
     PRINT_OFFSET(VM, selm.s.Tss);
-    PVM pVM;
+    PVM pVM = NULL; NOREF(pVM);
     if ((RT_OFFSETOF(VM, selm.s.Tss) & PAGE_OFFSET_MASK) > PAGE_SIZE - sizeof(pVM->selm.s.Tss))
     {