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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/14/19 11:18 AM, Jonathon Jongsma wrote:

>>
>> Note that we do have syntax-check for this, it had to be loosened in:
>> commit ac41ffb8fa1235c226377397842c6c240a677d87
>> Author:     Cole Robinson <crobinso@xxxxxxxxxx>
>> AuthorDate: 2019-04-03 17:45:02 -0400
>> Commit:     Cole Robinson <crobinso@xxxxxxxxxx>
>> 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@xxxxxxxxxx>
>>     Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
>>
>> But currently it accepts both variants.
>>
>> Jano
> 
> Out of curiosity, do we want to use #pragma once in the public headers
> as well, or only the internal ones?
> 

Only internal. Our public headers (in theory) should be usable from any
non-gcc/clang C99 compiler.  #pragma once is a non-portable compiler
extension that happens to be portable to the only two compilers we admit
for building libvirt itself, but that does not imply that we are going
to force other projects using the public headers to do the same.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux