[Spice-devel] [PATCH 3.10/12] Remove a couple single-use static functions

Christophe Fergeau cfergeau at redhat.com
Fri Oct 30 04:43:09 PDT 2015


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151030/785d12e1/attachment.sig>


More information about the Spice-devel mailing list