Re: [PATCH v3 03/31] Add parse_pathspec() that converts cmdline args to struct pathspec
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Jan 14, 2013, 01:11 UTC
- Message-ID
- <CACsJy8DBcmFDBYXQDuS_8u5sA+1z5GUpBN1kv8KYcwF9gg=mFg@mail.gmail.com>
- In-Reply-To
- <CANiSa6icv7V7hoEzHQT0mgqjCDcSkuLvZ2M=6Q5gp6NcXJ20jQ@mail.gmail.com>
On Mon, Jan 14, 2013 at 7:05 AM, Martin von Zweigbergk <martinvonz@gmail.com> wrote:
Show 12 quoted lines
> On Sun, Jan 13, 2013 at 4:35 AM, Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
>> +static void parse_pathspec(struct pathspec *pathspec,
>> + unsigned magic_mask, unsigned flags,
>> + const char *prefix, const char **argv)
>> +{
>> + struct pathspec_item *item;
>> + const char *entry = *argv;
>> ...
>> + for (i = 0; i < n; i++) {
>> + const char *arg = argv[i];
>
> Nit: "*argv" was assigned to "entry" above. Reassign that variable instead?Yeah. Thanks for catching.
-- Duy