[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?: |
Date: |
Thu, 12 Jul 2012 20:28:57 +0000 |
On Wed, Jul 11, 2012 at 12:54 PM, Kevin Wolf <address@hidden> wrote:
> Am 08.07.2012 14:09, schrieb Andreas Schwab:
>> address@hidden writes:
>>
>>> + pstrcpy(bs->backing_format, sizeof(bs->backing_format),
>>> + backing_fmt ? backing_file : "");
>>
>> s/backing_file/backing_fmt/
>
> Which is why such changes are probably a bad idea. Even more so if they
> aren't scripted.
Maybe your patches are perfect from day one, but all patches can be
buggy. Review should catch some of the bugs, others may be found
later. It's not possible to script this because expr1 may have side
effects.
>
> Does this patch improve anything? Last time I checked, qemu only
> compiled on gcc anyway.
It improves C99 compliance. GCC extensions should not be used unless
absolutely required. In the future, it should be possible to compile
QEMU with any C compiler, AREG0 patches remove the biggest obstacle.
>
> Kevin
- [Qemu-devel] [PATCH 0/5] portability patches, blauwirbel, 2012/07/08
- [Qemu-devel] [PATCH 2/5] Avoid returning void, blauwirbel, 2012/07/08
- [Qemu-devel] [PATCH 3/5] Use __asm__ instead of asm or __asm, blauwirbel, 2012/07/08
- [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, blauwirbel, 2012/07/08
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Andreas Schwab, 2012/07/08
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Blue Swirl, 2012/07/08
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Kevin Wolf, 2012/07/11
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Peter Maydell, 2012/07/11
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Wei-Ren Chen, 2012/07/11
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Blue Swirl, 2012/07/12
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Peter Maydell, 2012/07/12
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Blue Swirl, 2012/07/13
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Kevin Wolf, 2012/07/13
- Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Blue Swirl, 2012/07/13
Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:, Markus Armbruster, 2012/07/09
[Qemu-devel] [PATCH 5/5] Avoid redefining inline, blauwirbel, 2012/07/08
[Qemu-devel] [PATCH 4/5] Avoid unportable %m format, blauwirbel, 2012/07/08