[Spice-devel] [PATCH v2 11/13] Add mention of header guards

Christophe Fergeau cfergeau at redhat.com
Wed Feb 14 13:29:58 UTC 2018


On Thu, Feb 08, 2018 at 03:58:46AM -0500, Frediano Ziglio wrote:
> > +
> > +[source,h]
> > +---
> > +#ifndef MY_MODULE_H
> > +#define MY_MODULE_H
> > +
> > +...
> > +
> > +#endif /* MY_MODULE_H */
> > +---
> > +
> 
> Are we suggesting C style only comment or is clear the is not important?
> (I'm just drinking my first coffee this morning)

spice-server is consistently using #endif /* */ for header guards, I'd
rather this stays consistent. I'd say use /* */ in C projects, and // in
C++ ones.

Christophe
-------------- 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/20180214/5f06d71a/attachment.sig>


More information about the Spice-devel mailing list