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(a)redhat.com>
and pushed.
Jano