[libvirt] [PATCH 00/19] Use '#pragma once' in headers

[Ján Tomko]

On Fri, Jun 07, 2019 at 03:20:11PM -0500, Jonathon Jongsma wrote:
>Hi all,

Welcome,

>
>I'm trying to get up to speed with libvirt code,

please if you have something you wish we mentioned in our HACKING page
or elsewhere to make the first contribution easier, let us know.

>so I decided to start out with a slightly
>"mindless" task. I grabbed the '#pragma once' task from
>https://wiki.libvirt.org/page/BiteSizedTasks. This series covers the first part of the conversion. I
>chatted with Daniel Berrange briefly on IRC before starting, and he suggested that I should try to
>keep patches under 100k, and maybe each driver's source files as a single patch.
>
>I had a bit of difficulty deciding where to split the src/conf/ and src/util/ directories. So some
>of the files with more changes ended up being single-file patches. I tried to combine some related
>source files into patches, but at a certain point I ran out of obvious files to combine, so I
>combined a bunch of smaller miscellaneous files into one larger patch. It may not be ideal, but it's
>the best I could come up with. Alternative suggestions welcome.
>
>I'll follow up with the remaining changes if this division looks acceptable.
>
>Jonathon Jongsma (19):
>  Use #pragma once in driver headers
>  src/access: use #pragma once in headers
>  src/admin: use #pragma once in headers
>  src/bhyve: use #pragma once in headers
>  conf: domain: use #pragma once in headers
>  conf: network: use #pragma once in headers
>  conf: nwfilter: use #pragma once in headers
>  conf: node device: use #pragma once in headers
>  conf: storage: use #pragma once in headers
>  conf: misc: use #pragma once in headers
>  src/cpu: use #pragma once in headers
>  src/datatypes.h: use #pragma once
>  src/esx: use #pragma once in headers
>  src/hyperv: use #pragma once in headers
>  src/interface: use #pragma once in headers
>  src/internal.h: use #pragma once
>  src/libvirt_internal.h: use #pragma once
>  src/libxl: use #pragma once in headers
>  src/logging: use #pragma once in headers
>
> src/access/viraccessdriver.h         |  10 +-
> src/access/viraccessdrivernop.h      |   7 +-
[...]
> src/logging/log_daemon_dispatch.h    |   7 +-
> src/logging/log_handler.h            |   9 +-
> src/logging/log_manager.h            |   9 +-
> 117 files changed, 928 insertions(+), 1296 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko at redhat.com>
and pushed.

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190613/22454d3d/attachment-0001.sig>