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

Reply via email to