Re: [PATCH] greybus: Fix coding stye error
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
- Subject: Re: [PATCH] greybus: Fix coding stye error
- From: Johan Hovold <johan@xxxxxxxxxx>
- Date: Fri, 27 Jan 2017 14:08:06 +0100
- In-reply-to: <20170127124721.GP4201@mwanda>
- References: <1485511881-8693-1-git-send-email-mrossibellom@gmail.com> <20170127122930.GL4149@mwanda> <20170127123905.GF3560@localhost> <20170127124721.GP4201@mwanda>
- User-agent: Mutt/1.5.24 (2015-08-30)
On Fri, Jan 27, 2017 at 03:47:21PM +0300, Dan Carpenter wrote:
> The name sucks is what I'm saying. Please fix it eventually.
I disagree. The name uses a common prefix that reflects the object it is
working on. This should not be changed. As it is currently named, the
function is also self-documenting. Trying to abbreviate the name just to
meet a 80 col limit (when there are alternatives) is just silly.
Johan
_______________________________________________
greybus-dev mailing list
greybus-dev@xxxxxxxxxxxxxxxx
https://lists.linaro.org/mailman/listinfo/greybus-dev
[Index of Archives]
[Asterisk App Development]
[PJ SIP]
[Gnu Gatekeeper]
[IETF Sipping]
[Info Cyrus]
[ALSA User]
[Fedora Linux Users]
[Linux SCTP]
[DCCP]
[Gimp]
[Yosemite News]
[Deep Creek Hot Springs]
[Yosemite Campsites]
[ISDN Cause Codes]
[Asterisk Books]