On Fri, April 4, 2008 3:19 am, Phil Dibowitz wrote: > Stephen Warren wrote: >> Here's an updated version of the USB patch that works with WinHID on >> Windows. I had to fix: >> >> HID_WriteReport: MS compiler won't allow creation of a variable-sized >> array on the stack, so I made the code new/delete it instead. > > You changed if (Write...) to an ret = Write(..); if (err), which seemed > odd to me, so I changed it back.
Oops. I originally had the delete[] call between those two lines, but of course that broke, because the code uses asynchronous IO, so the delete needs to happen after the event wait. I evidently forgot revert the change when I did that. Did you also remove the ret variable? I added that just for that change, I think. ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace _______________________________________________ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel