[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC
From: |
Filip Navara |
Subject: |
Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC |
Date: |
Wed, 24 Jun 2009 20:59:11 +0200 |
On Wed, Jun 24, 2009 at 8:54 PM, Andreas Färber<address@hidden> wrote:
>
> Am 24.06.2009 um 20:25 schrieb Jamie Lokier:
>
>> Andreas Färber wrote:
>>>
>>> Am 21.06.2009 um 01:25 schrieb Jamie Lokier:
>>>
>>>> François Revol wrote:
>>>>>>
>>>>>> On 06/20/2009 05:31 PM, Anthony Liguori wrote:
>>>>>>>
>>>>>>> Please [...] use C-style comments instead
>>>>>>> of C++ style comments. [...]
>>>>>>
>>>>>> What do we have against C++ style comments?
>>>>>
>>>>> They are not valid in C code :P
>>>>
>>>> Wrong.
>>>>
>>>> Valid in GNU C for about 14 years, and ANSI/ISO C for about 10
>>>> years...
>>>>
>>>> I'll be surprised if any C compiler used on any open source operating
>>>> system, or any major commercial C compiler, doesn't accept them.
>>>
>>> Gladly even ten-year-old GCC 2.95.x supports 'em. But even a recent
>>> GCC does not when compiled with -ansi, which is why system headers
>>> still mustn't use them.
>>
>> This is wrong too. They can be used in system headers.
>
> No, that is wrong. freetype2 definitely complained when Haiku had C99
> comments in its POSIX system headers. ;)
#pragma GCC system_header
Best regards,
Filip Navara
- [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, m a, 2009/06/19
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Anthony Liguori, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Avi Kivity, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, François Revol, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Avi Kivity, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Jamie Lokier, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Andreas Färber, 2009/06/21
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Jamie Lokier, 2009/06/24
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Andreas Färber, 2009/06/24
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC,
Filip Navara <=
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Jamie Lokier, 2009/06/24
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Anthony Liguori, 2009/06/20
- Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Avi Kivity, 2009/06/21
Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, Christoph Hellwig, 2009/06/20
Re: [Qemu-devel] [PATCH] replace O_SYNC with O_FSYNC, G 3, 2009/06/20