Re: [PATCH 3.10/12] Remove a couple single-use static functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 30, 2015 at 03:44:05AM -0400, Frediano Ziglio wrote:
> OT: I don't fully agree with the reasoning. Today compilers are really
> good inlining single used static functions. Putting even some lines in
> small functions can improve readability if function name is well choosed,
> future maintenance and type safety.

Same feeling here, save for red_cursor_marshall_inval() in that patch.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]