[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in conf
From: |
Chen Gang |
Subject: |
Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator() |
Date: |
Mon, 31 Mar 2014 21:12:11 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 03/31/2014 09:01 PM, Peter Maydell wrote:
> On 31 March 2014 13:53, Chen Gang <address@hidden> wrote:
>> On 03/31/2014 08:38 PM, Markus Armbruster wrote:
>>> Chen Gang <address@hidden> writes:
>>>
>>>> Hello Maintainers:
>>>>
>>>> If it is necessary to send patch v2 by me, please let me know, I
>>>> will/should send.
>>>
>>> Not a maintainer, but if you send a v2 with an improved commit message,
>>> I'll R-by it, which can only help getting it merged.
>>>
>>
>> I guess your meaning is "not quite necessary" (for me, minor useful
>> patches almost like spam). So if sending patch v2 is really required,
>> please let me know, thanks.
>
> Basically, asking a maintainer to make changes to a patch
> as they apply it is asking them to do extra work beyond
> what they would normally do. Sometimes people will agree
> to do this, but in general it's better just to send a fixed
> version of the patch yourself.
>
> (I've cc'd qemu-trivial since that's probably the best tree
> to take this patch.)
>
OK, thanks. And excuse me, my English is not quite well, I guess, I
misunderstood the original replier's meaning. I will/should send patch
v2 for it within this week (2014-04-06).
Next, when I send trivial patches, I will/should cc to qemu-trivial. I
guess, most of my future patches will be trivial patches (and for me,
trivial != minor).
Thanks.
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed
- [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/26
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Marcel Apfelbaum, 2014/03/27
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Markus Armbruster, 2014/03/27
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/27
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/30
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Markus Armbruster, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(),
Chen Gang <=
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Peter Maydell, 2014/03/31
- Re: [Qemu-devel] [PATCH] vl.c: use 'break' instead of 'continue' in configure_accelerator(), Chen Gang, 2014/03/31