Subject: Re: [PATCH 2/6] Access reference caches only through new function get_cached_refs().

From: Michael Haggerty <[email-masked]>
Date: Tue, 23 Aug 2011 06:21:55 +0200

On 08/15/2011 12:12 AM, Junio C Hamano wrote:
> Michael Haggerty <[email-masked]> writes:
>> +	if (! submodule)
> 
> (style) lose the SP before "submodule".

Will be fixed in re-roll.

>> -	if (!refs->did_packed || submodule) {
>> +	if (!refs->did_packed) {
>> +		const char *packed_refs_file;
>> +		if (submodule)
>> +			packed_refs_file = git_path_submodule(submodule, "packed-refs");
>> +		else
>> +			packed_refs_file = git_path("packed-refs");
>>  		FILE *f = fopen(packed_refs_file, "r");
> 
> decl-after-statement.

Will be fixed.

On 08/15/2011 12:21 AM, Junio C Hamano wrote:
> Michael Haggerty <[email-masked]> writes:
>> +struct cached_refs *create_cached_refs()
>
> struct cached_refs *create_cached_refs(void)

Will be fixed.

Thanks for your feedback.

Michael

-- 
Michael Haggerty
[email-masked]
http://softwareswirl.blogspot.com/