Thread (8 messages) flat view 8 messages, 3 authors, 2011-06-13

RE: [PATCH] usb:gadget: use min_t() macro instead of min()

From: Alan Stern <stern@rowland.harvard.edu>
Date: 2011-06-13 19:03:46
Also in: linux-arm-msm, linux-omap, lkml

On Mon, 13 Jun 2011, Tanya Brokhman wrote:
quoted
The change I suggested involved replacing two typecasts with a single
min_t.  All (or almost all) the places this patch touches currently
contain only one typecast, so the motivation for changing them is a lot
weaker.
You're right. So what's the final call on this one? Do you think it can be
merged or you prefer not change anything? I personally think the code looks
nicer using min_t instead of min with casting but that's just my opinion and
of course there are arguments against this patch.
I don't care either way.  It's up to you and Felipe.

Alan Stern
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help