Re: kernel32/tests : added tests for copyfileex try7

2013-02-01 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=24239 Your paranoid

Re: [PATCH] cryptui: use add_oid_to_usage correctly (Coverity)

2013-02-01 Thread Juan Lang
Hi Marcus, -add_oid_to_usage(usage, ptr); +usage = add_oid_to_usage(usage, ptr); This looks fine, but would you mind making the same change on line 337? Thanks, --Juan

Re: [PATCH] cryptui: use add_oid_to_usage correctly (Coverity)

2013-02-01 Thread Juan Lang
On Fri, Feb 1, 2013 at 3:45 PM, Juan Lang juan.l...@gmail.com wrote: Hi Marcus, -add_oid_to_usage(usage, ptr); +usage = add_oid_to_usage(usage, ptr); This looks fine, but would you mind making the same change on line 337? Actually, perhaps I hit sent too early. If