Re: [PATCH 08/19] iwlegacy: Return directly if allocation fails in il_eeprom_init()
From: Julian Calaby <hidden>
Date: 2016-03-19 13:17:14
From: Julian Calaby <hidden>
Date: 2016-03-19 13:17:14
Hi Dan, On Sat, Mar 19, 2016 at 1:53 AM, Dan Carpenter [off-list ref] wrote:
On Sat, Mar 19, 2016 at 01:33:16AM +1100, Julian Calaby wrote:quoted
Hi Markus, On Sat, Mar 19, 2016 at 12:41 AM, SF Markus Elfring [off-list ref] wrote:quoted
quoted
Also remove an unused label.Is such a commit message a bit too short?I don't think so, but I don't decide these sorts of things. Does anyone else have an opinion on this?This is certainly not something where we need to redo the patch but since you asked for the future. A lot of web archives put the subject and the body of the email far apart: http://marc.info/?l=linux-wireless&m=145826783627087&w=2 lkml.org is the same way. It makes these types of patch descriptions awkward.
Makes sense, I'll try to avoid descriptions like this in the future. Thanks, -- Julian Calaby Email: julian.calaby@gmail.com Profile: http://www.google.com/profiles/julian.calaby/