Re: [PATCH v3 5/7] path-walk: visit tags and cached objects

>> +		/* Navigate annotated tag object chains. */
>> +		while (obj->type == OBJ_TAG) {
>> +			struct tag *tag = lookup_tag(info->revs->repo, &obj->oid);
>> +			if (!tag)
>> +				break;
> 
> Same here as previous comments, is this an error that we should rather
> report?

> [snip]
>> +	if (tagged_blobs) {
>> +		if (tagged_blobs->oids.nr) {
>> +			const char *tagged_blob_path = "/tagged-blobs";
>> +			tagged_blobs->type = OBJ_BLOB;
>> +			push_to_stack(ctx, tagged_blob_path);
>> +			strmap_put(&ctx->paths_to_lists, tagged_blob_path, tagged_blobs);
>> +		} else {
>> +			oid_array_clear(&tagged_blobs->oids);
>> +			free(tagged_blobs);
>> +		}
>> +	}
>> +	if (tags) {
>> +		if (tags->oids.nr) {
>> +			const char *tag_path = "/tags";
>> +			tags->type = OBJ_TAG;
>> +			push_to_stack(ctx, tag_path);
>> +			strmap_put(&ctx->paths_to_lists, tag_path, tags);
>> +		} else {
>> +			oid_array_clear(&tags->oids);
>> +			free(tags);
>> +		}
>> +	}
>> +}
> 
> So this is kind of curious. Does that mean that a file named
> "tagged-blobs" would be thrown into the same bag as a tagged blob? Or
> are these special due to the leading "/"?