Re: [ibm-acpi-devel] [PATCH 8/14] drivers/platform/x86/thinkpad_acpi.c: delete double assignment

2010-11-24 Thread Matthew Garrett
Applied, thanks. -- Matthew Garrett | mj...@srcf.ucam.org -- Increase Visibility of Your 3D Game App & Earn a Chance To Win $500! Tap into the largest installed PC base & get more eyes on your game by optimizing for Inte

Re: [ibm-acpi-devel] [PATCH 8/14] drivers/platform/x86/thinkpad_acpi.c: delete double assignment

2010-10-27 Thread Henrique de Moraes Holschuh
On Wed, 27 Oct 2010, Dan Carpenter wrote: > On Tue, Oct 26, 2010 at 11:09:08PM -0200, Henrique de Moraes Holschuh wrote: > > On Tue, 26 Oct 2010, Julia Lawall wrote: > > > From: Julia Lawall > > > > > > Delete successive assignments to the same location. > > > > There are often results of mismer

Re: [ibm-acpi-devel] [PATCH 8/14] drivers/platform/x86/thinkpad_acpi.c: delete double assignment

2010-10-26 Thread Henrique de Moraes Holschuh
On Tue, 26 Oct 2010, Henrique de Moraes Holschuh wrote: > On Tue, 26 Oct 2010, Julia Lawall wrote: > > From: Julia Lawall > > > > Delete successive assignments to the same location. > > There are often results of mismerges or other assorted screw ups, often > done by the maintainer itself (e.g.

Re: [ibm-acpi-devel] [PATCH 8/14] drivers/platform/x86/thinkpad_acpi.c: delete double assignment

2010-10-26 Thread Henrique de Moraes Holschuh
On Tue, 26 Oct 2010, Julia Lawall wrote: > From: Julia Lawall > > Delete successive assignments to the same location. There are often results of mismerges or other assorted screw ups, often done by the maintainer itself (e.g. me in thinkpad-acpi's case) during development. The patch is correct,