Messages in this thread | ![/](/images/icornerl.gif) | | From | "Luis R. Rodriguez" <> | Date | Mon, 29 Nov 2010 14:42:50 -0800 | Subject | Re: [ath9k-devel] [PATCH wireless-next] ath: Rename ath_print to ath_debug |
| |
On Mon, Nov 29, 2010 at 2:41 PM, Felix Fietkau <nbd@openwrt.org> wrote: > On 2010-11-29 7:07 AM, Peter Stuge wrote: >> Luis R. Rodriguez wrote: >>> On Sun, Nov 28, 2010 at 3:53 PM, Joe Perches <joe@perches.com> wrote: >>> > Make the function name match the function purpose. >>> > ath_debug is a debug only facility. >>> > ath_print seems too generic a name for a debug only use. >>> >>> Nack, I don't see the point. >> >> The point is to improve readability. I like the patch. > And how exactly does this improve readability? Don't get me wrong, I > generally like to see more cleanups merged to the ath/ath9k drivers > (they do need it, after all). > > But in my opinion, simple renaming churn like this does nothing but > annoy people who want to get other work done at the same time. > Consider the large number of lines touched (and the potential merge > conflicts with other code because of that), relative to the microscopic > aesthetic gain (if any). > > Instead I'd like to see more cleanups that go beyond trivial function > renames.
Thanks for putting it so clearly. Agreed.
Luis
| ![\](/images/icornerr.gif) |