Changeset 62707 in vbox for trunk/src/VBox/HostDrivers/VBoxUSB/win/dev

Timestamp:

Jul 29, 2016 8:31:21 PM (8 years ago)

Author:

vboxsync

Message:

VBoxUsbRt.cpp: This cluelessness wrt assertion macros should be a reason for dismissal! The use of FUNCTION in the assertion message takes the prize, though. Cleaning up crap like this is such a stupid of time, so it'll have to wait.

File:

• trunk/src/VBox/HostDrivers/VBoxUSB/win/dev/VBoxUsbRt.cpp (modified) (1 diff)

trunk/src/VBox/HostDrivers/VBoxUSB/win/dev/VBoxUsbRt.cpp

@@ -594 +594 @@
     }
 
+/** @todo r=bird: Need to write a script for fixing these kind of clueless use
+ *        of AssertMsgFailed (into AssertMsgReturn).  The __FUNCTION__ is just
+ *        the topping it off - the assertion message includes function, file and
+ *        line number. Duh! */
     PUSB_CONFIGURATION_DESCRIPTOR pCfgDr = vboxUsbRtFindConfigDesc(pDevExt, uConfiguration);
     if (!pCfgDr)