Re: [ath9k-devel] [PATCH net-next 00/21] treewide: Use consistent api style for address testing

From
Pavel Roskin <hidden>
Date
2012-10-19 15:55:17
Also in
batman, bridge, linux-arm-kernel, linux-devicetree, linux-mips, linux-nfs, linux-rdma, linux-scsi, linux-um, lvs-devel, netdev, netfilter-devel
Reply to
Eric Dumazet, 2012-10-19
On Fri, 19 Oct 2012 09:04:14 +0200
Eric Dumazet [off-list ref] wrote:
Yes they are some names discrepancies, thats a big deal.

And we have alloc_skb() / kfree_skb() / skb_clone() 

Why not skb_alloc() / skb_free() / skb_clone() ?

Some people actually know current code by name of functions, they dont
want to change their mind and having to grep include files and git log
to learn the new names of an old function, especially when traveling
and using a laptop.
I agree.

Also, it makes sense to introduce a more consistent name for a function
when it's improved in some way and the callers need to be adjusted or
re-checked.

That way, the old name can be phased out as the code is made compatible
with the new function.

-- 
Regards,
Pavel Roskin
Thread (3 messages) 3 messages, 3 authors, last reply 2012-10-19
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help