|
From: | Stefan Weil |
Subject: | Re: [Qemu-devel] [PATCH 00/12] target-s390x: Several small fixes |
Date: | Fri, 27 May 2011 06:59:09 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.17) Gecko/20110424 Thunderbird/3.1.10 |
Am 26.05.2011 23:48, schrieb Andreas Färber:
Am 26.05.2011 um 00:17 schrieb Alexander Graf:On 26.05.2011, at 00:10, Peter Maydell wrote:On 25 May 2011 21:25, Stefan Weil <address@hidden> wrote:Feel free to combine patches if larger patches are preferred.I'd vote for combining at least 03..12, having nine patches all of which have the same summary line is a bit confusing :-)I actually like them as individual patches. Makes bisecting a lot easier :).Stefan, could you add the function name or some other discriminator to the subject then, please?Andreas
This would result in these subjects: 4 x target-s390x: Add missing tcg_temp_free_i64() in disas_a5 2 x target-s390x: Add missing tcg_temp_free_i64() in disas_s390_insn 1 x target-s390x: Add missing tcg_temp_free_i64() in disas_b2 1 x target-s390x: Add missing tcg_temp_free_i64() in do_mh 1 x target-s390x: Add missing tcg_temp_free_i64() in gen_jccSo there remain duplicate subjects. Adding the line number or a sequence number
would look somehow strange. The subjects are identical simply because the changes are very similar. If needed, git provides the unique identification. Maybe combining the patches which share the same function and add this function name to the subject would be a compromise which still allows bisecting. Cheers, Stefan
[Prev in Thread] | Current Thread | [Next in Thread] |