On 08/15/2011 12:12 AM, Junio C Hamano wrote:
> Michael Haggerty <mhagger@xxxxxxxxxxxx> 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 <mhagger@xxxxxxxxxxxx> writes:
>> +struct cached_refs *create_cached_refs()
>
> struct cached_refs *create_cached_refs(void)
Will be fixed.
Thanks for your feedback.
Michael
--
Michael Haggerty
mhagger@xxxxxxxxxxxx
http://softwareswirl.blogspot.com/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html