[MERGE][#107155] _get_vendor_by_inspection incorrectly determines "plink" to be the executable
Alexander Belchenko
bialix at ukr.net
Sun Sep 2 19:00:15 BST 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Martin Albisetti пишет:
> On 9/2/07, Alexander Belchenko <bialix at ukr.net> wrote:
>> ^-- arg is really bad name here.
>> and
>> + :param arg: Argument executed.
>> too. Actually executed command or command line, "argument" looks slightly odd.
>> Better to change this param to "args" as Martin Pool suggested, or "cmd_line".
>
> Initially I chose "arg" instead of "args" because only one "command"
> is passed, and not multiple ones like in the array.
>
> I agree a "args" (or even "cmd_line" is better, although it should be
> changed in the loop then too. How would I go on by changing this?
> Send another patch?
No, tweaked version with s/arg/args/ already merged.
I missed it at first time.
- --
[µ]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFG2vovzYr338mxwCURAu/HAJ9q7VPkHwW+styictcKkGvputSDLQCfaL9E
XeLZNs8eOnnsI4cTg4Af4OA=
=zX1Y
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list