Changeset 44707 in vbox for trunk/include/VBox/vmm

Timestamp:

Feb 15, 2013 1:48:58 AM (12 years ago)

Author:

vboxsync

Message:

PDMIVMMDEVPORT::pfnRequestDisplayChange: Attempted documenting "new" members - whoever added these please check and remove comments in the parenthesis. (Also, please don't forget next time.)

File:

• trunk/include/VBox/vmm/pdmifs.h (modified) (1 diff)

trunk/include/VBox/vmm/pdmifs.h

@@ -2066 +2066 @@
      * @param   cBits           Bits per pixel (0 = do not change).
      * @param   idxDisplay      The display index.
+     * @param   xOrigin         The X coordinate of the lower left corner of the
+     *                          display.  (Presumably, as the lazy coder
+     *                          adding this parameter didn't bother to document it.)
+     * @param   yOrigin         The Y coordinate of the lower left corner of the
+     *                          display.  (Presumably, as the lazy coder adding
+     *                          this parameter didn't bother to document it.)
+     * @param   fEnabled        Whether the display is enabled or not. (Guessing
+     *                          again.)
+     * @param   fChangeOrigin   Whether the display origin point changed. (Guess)
+     * @todo r=bird: Any reason why this interface is using uint32_t and not int32_t
+     *       for xOrigin and yOrigin? The Main side is using LONG there, while the
+     *       cx/cy are ULONG.
      */
     DECLR3CALLBACKMEMBER(int, pfnRequestDisplayChange,(PPDMIVMMDEVPORT pInterface, uint32_t cx,
                          uint32_t cy, uint32_t cBits, uint32_t idxDisplay,
-                         uint32_t cxOrigin, uint32_t cyOrigin, bool fEnable, bool fChangeOrigin));
+                         uint32_t xOrigin, uint32_t yOrigin, bool fEnable, bool fChangeOrigin));
 
     /**