Thread (48 messages) 48 messages, 7 authors, 2016-04-26

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

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/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help