Bug#687160: patch review

2012-10-10 Thread Dmitrijs Ledkovs
On 10/10/12 01:37, dann frazier wrote: hey Dmitrijs! I prepared a similar fix for this a couple of weeks ago that has been in testing at a customer site. Looks like we crossed paths, but our fixes are very similar. My patch has been working well for my test case, and I've verified yours does

Bug#687160: patch review

2012-10-09 Thread dann frazier
hey Dmitrijs! I prepared a similar fix for this a couple of weeks ago that has been in testing at a customer site. Looks like we crossed paths, but our fixes are very similar. My patch has been working well for my test case, and I've verified yours does too. However, I've a question on your patch