Re: [PATCH v5 28/28] count-objects: report unused files in $GIT_DIR/repos/...
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Mar 9, 2014, 11:25 UTC
- Message-ID
- <CACsJy8DGSTYxam7Xws=y7as+h7Mppj9yXr_FwDywQxG8p3dr=g@mail.gmail.com>
- In-Reply-To
- <CAPig+cQpzYo_VzJfjMCevYm4__bHPSaFLNZSv2MFMA3ondJoSQ@mail.gmail.com>
On Sun, Mar 9, 2014 at 3:21 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 25 quoted lines
> On Fri, Mar 7, 2014 at 9:48 PM, Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
>> In linked checkouts, borrowed parts like config is taken from
>> $GIT_COMMON_DIR. $GIT_DIR/config is never used. Report them as
>> garbage.
>>
>> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
>> ---
>> diff --git a/path.c b/path.c
>> index ddb5962..5a7dc45 100644
>> --- a/path.c
>> +++ b/path.c
>> @@ -4,6 +4,7 @@
>> #include "cache.h"
>> #include "strbuf.h"
>> #include "string-list.h"
>> +#include "dir.h"
>>
>> static int get_st_mode_bits(const char *path, int *mode)
>> {
>> @@ -91,9 +92,9 @@ static void replace_dir(struct strbuf *buf, int len, const char *newdir)
>> }
>
> Do you want to add a comment here explaining what the "!" prefix on
> some entries means, or is it sufficiently self-evident to anyone
> looking at the consumers of this array?I was hoping it was clear from the patch how this "!" was used (or "/" in the patch that introduces common_list[]). But if any reader thinks otherwise, I'm happy to add a comment.
-- Duy