Just applied 3&4 but dropped 1&2. 1&2 produce sign conversion warnings 
in MSVC, that seem to be justified since we're comparing _packet with 
transfer->num_iso_packets, that is defined as an int. I don't quite get 
why we would want to switch to unsigned there.

Also, could you please try to merge multiple one liners into a single 
patch? Even if I had to reject 2 of them, applying multiple one liners 
that deal with the same warning IN A ROW is a bit of a PITA, and 
requires more time than I'd like to spend on the task.

Else, I'll probably let you apply these changes to mainline on your own 
(by following [1]) ;)

Regards,

/Pete

[1] 
https://github.com/libusbx/libusbx/wiki/Maintainers%27-Corner#wiki-Pushing_changes_to_libusbx_mainline

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnnow-d2d
_______________________________________________
libusbx-devel mailing list
libusbx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libusbx-devel

Reply via email to