[Spice-devel] Coding style and naming conventions for C++

[Victor Toso]

On Tue, Feb 06, 2018 at 04:40:18AM -0500, Frediano Ziglio wrote:
> > 
> > On Tue, 2018-01-30 at 11:50 +0100, Christophe de Dinechin wrote:
> > > 
> > > Let’s first share our preference on existing styles to see
> > > if we agree on anything…
> > > As for me, I have a slight preference for the LLVM coding
> > > style, but I made modifications in my own clang-format
> > > files.
> > 
> > Ok, so since nobody really expressed an opinion here, I'm
> > inclined to proceed with my original proposal. Frediano
> > seemed to agree, is it so, Frediano?

I don't have anything against the proposal...

> > 
> > The LLVM coding style still is an alternative, but I still
> > consider camel_case_function_names better for consistency
> > with our C code.
> > 
> > Lukas
> 
> I would like to see these style in a documentation.
> I think the quickest way is to integrate these notes in
> spice-server docs/spice_style.txt and put a simple txt document
> in spice-streaming-agent to refer to this. This for various
> reasons:

Agreed.

> - spice-server already "compile" this document in HTML;
> - the document is used for https://www.spice-space.org/spice-project-coding-style-and-coding-conventions.html
>   page so you'll have the web page for free;
> - the style is mainly an extension of the server style, copying all the
>   style to the new repository would be a big effort to maintain in sync
>   (another option would be to put in the new repository only the additions/
>   exception but this won't solve previous points)
> 
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20180206/f8d58f03/attachment.sig>