On Wed, Jun 12, 2013 at 4:47 PM, Célestin Matte
<celestin.ma...@ensimag.fr> wrote:
> Le 12/06/2013 22:34, Célestin Matte a écrit :
>>> Célestin Matte <celestin.ma...@ensimag.fr> writes:
>> Actually, there still is a problem: the "defined(@array)" (introduced
>> with the change in [21/31]) produces a warning.
>> Benoit also seems to have problems with this patch... I'm waiting for
>> his feedback.
>>
> There is indeed a bug. In patch [21/31], at line 163, it should be
> if (!defined $cmd[0]) {
> instead of
> if (!defined @cmd) {
>
> Should I send yet another version of this series of patch?

Assuming you don't discover more problems, for such a small change in
a lengthy patch series, it might make sense to send a simple "fixup!"
[*1*] patch and ask if Junio would be willing to squash it into v5.

[*1*]: see fixup! in git/Documentation/git-rebase.txt
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to