[libvirt] [PATCH 00/19] Use '#pragma once' in headers
Ján Tomko
jtomko at redhat.com
Fri Jun 14 06:07:32 UTC 2019
On Thu, Jun 13, 2019 at 10:23:30AM -0500, Eric Blake wrote:
>On 6/13/19 10:06 AM, Ján Tomko wrote:
>
>>> 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
>>>
>
>
>Does this series add a syntax-check rule to prevent reversion back to
>older styles when new files are introduced? For example, I now have to
>rebase my pending incremental backup work on top of this, which adds new
>files, but without a syntax-check rule addition, I can't quickly
>identify which files I plan to add that need a tweak.
>
Note that we do have syntax-check for this, it had to be loosened in:
commit ac41ffb8fa1235c226377397842c6c240a677d87
Author: Cole Robinson <crobinso at redhat.com>
AuthorDate: 2019-04-03 17:45:02 -0400
Commit: Cole Robinson <crobinso at redhat.com>
CommitDate: 2019-04-04 18:42:09 -0400
build-aux: header-ifdef: Handle #pragma once
If we see it, skip all remaining header guard checks
Acked-by: Eric Blake <eblake at redhat.com>
Signed-off-by: Cole Robinson <crobinso at redhat.com>
But currently it accepts both variants.
Jano
>>
>> Reviewed-by: Ján Tomko <jtomko at redhat.com>
>> and pushed.
>Thus, it would be a nice followup patch for you to work on.
>
>--
>Eric Blake, Principal Software Engineer
>Red Hat, Inc. +1-919-301-3226
>Virtualization: qemu.org | libvirt.org
>
-------------- 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/20190614/8c92a4ad/attachment-0001.sig>
More information about the libvir-list
mailing list