[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure |
Date: |
Tue, 22 May 2012 11:34:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
Am 22.05.2012 11:05, schrieb Jim Meyering:
> Kevin Wolf wrote:
>> A patch replacing tabs by spaces isn't really the kind of patches that
>> we would want to avoid during freeze. It's easy enough to check with git
>> diff -w that it doesn't change anything semantically.
>
> That makes sense, so I've posted two patches:
>
> 1) two patches: one replacing all leading TABs with equivalent spaces,
> then the above patch
>
> Note however, that envlist.c must predate checkpatch.pl, since
> that patch provokes numerous warnings about style issues:
Quite possible. Most files in qemu will still contain some violations.
> (hmm... I've just noticed these first two, which seem to suggest
> that even for comments I should remove the TABs. Is it worth
> a V2 to fix those two lines? Or are they false positives, since
> they're not really "code" indentation? )
We don't use tabs at all where possible. There shouldn't be much more
exceptions than the Makefiles.
Kevin
- [Qemu-devel] [PATCH 0/3] unchecked uses of strdup, jim, 2012/05/15
- [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, jim, 2012/05/15
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Blue Swirl, 2012/05/19
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Jim Meyering, 2012/05/21
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Blue Swirl, 2012/05/21
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Kevin Wolf, 2012/05/22
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Jim Meyering, 2012/05/22
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH 1/3] envlist.c: handle strdup failure, Jim Meyering, 2012/05/22
[Qemu-devel] [PATCH 3/3] sparc: use g_strdup in place of unchecked strdup, jim, 2012/05/15
[Qemu-devel] [PATCH 2/3] scsi, pci, qdev, isa-bus, sysbus: don't let *_get_fw_dev_path return NULL, jim, 2012/05/15
- Re: [Qemu-devel] [PATCH 2/3] scsi, pci, qdev, isa-bus, sysbus: don't let *_get_fw_dev_path return NULL, Paolo Bonzini, 2012/05/15
- Re: [Qemu-devel] [PATCH 2/3] scsi, pci, qdev, isa-bus, sysbus: don't let *_get_fw_dev_path return NULL, Kevin Wolf, 2012/05/15