Re: [PATCH] Hash name is SHA-1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch should probably only change sha1 to SHA-1 and not reformat
the initialisation of _usage arrays or the comments (new line before
first line of comment).  If the reformatting is desired it should be a
separate patch imho.  Here is what I found:

Horst H. von Brand <vonbrand@xxxxxxxxxxxx> wrote:
> +head this is "refs/heads/master".  The two sha1 are the object names
> +called before the refname is updated, so either sha1-old is 0{40}
> +(meaning there is no such ref yet), or it should match what is
> +recorded in refname.
sha1 -> SHA-1?

> -static const char commit_tree_usage[] = "git-commit-tree <sha1> [-p <sha1>]* < changelog";
> +static const char commit_tree_usage[] = 
> +       "git-commit-tree <sha1> [-p <sha1>]* < changelog";
sha1 -> SHA-1?

> -static const char read_tree_usage[] = "git-read-tree (<sha> | [[-m [--aggressive] | --reset | --prefix=<prefix>] [-u | -i]] [--exclude-per-directory=<gitignore>] <sha1> [<sha2> [<sha3>]])";
> +static const char read_tree_usage[] = 
> +       "git-read-tree (<sha> | "
> +       "[[-m [--aggressive] | --reset | --prefix=<prefix>] "
> +       "[-u | -i]] [--exclude-per-directory=<gitignore>] "
> +       "<sha1> [<sha2> [<sha3>]])";
sha1 -> SHA-1, dunno what to do about the sha2 and sha3 :)
There is also a space at eol.

The following have a space at eol and/or spaces instead of tabs for
indenting.
> -	/* New file in the index: it might actually be different in
> +	/* 
> +         * New file in the index: it might actually be different in

> -	/* dst is recorded as a modification of src.  Are they so
> +	/* 
> +         * dst is recorded as a modification of src.  Are they so

> -/* We internally use unsigned short as the score value,
> +/* 
> + * We internally use unsigned short as the score value,

>   * and rely on an int capable to hold 32-bits.  -B can take
> -			/* When cloning, it is not unusual to have
> +			/* 
> +                         * When cloning, it is not unusual to have

> -	/* If there is data present from a previous transfer attempt,
> -	   resume where it left off */
> +	/* 
> +         * If there is data present from a previous transfer attempt,
> +         * resume where it left off 
> +         */

> -		/* Input is non-recursive ls-tree output format
> -		 * mode SP type SP sha1 TAB name
> +		/* 
> +                 * Input is non-recursive ls-tree output format
> +		 * mode SP type SP SHA-1 TAB name
-
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]