development/git
development/git
1
0
Fork 0
mirror of https://github.com/git/git synced 2024-07-04 16:48:40 +00:00
Code Issues Projects Releases Wiki Activity

rename/copy score parsing updates.

Browse Source 
Better variant, which handles stuff like "4.5%" and rejects
"192.168.0.1".  Additionally, make sure numbers are unsigned (I'm making
them unsigned long just for the hell of it), to make sure that
artificial wraparound scenarios don't cause harm.

	-hpa

[jc: with this, -M100 changes its meaning back to 10%.  People
wanting to say "pure renames only" should now say -M100% or
-M1.0; sounds a bit like an earthquake, but arguably things are
more consistent this way ;-)]

Signed-off-by: Junio C Hamano <junkio@cox.net>
This commit is contained in:
H. Peter Anvin 2005-11-21 14:17:12 -08:00 committed by Junio C Hamano
parent f35230fb11
commit 1b1480ff6a
1 changed files with 21 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
diff.c
View File

@ -838,29 +838,38 @@ int diff_opt_parse(struct diff_options *options, const char **av, int ac)
static int parse_num(const char **cp_p) static int parse_num(const char **cp_p)
{ {
int num, scale, ch, cnt; unsigned long num, scale;
int ch, dot;
const char *cp = *cp_p; const char *cp = *cp_p;
cnt = num = 0; num = 0;
scale = 1; scale = 1;
while ('0' <= (ch = *cp) && ch <= '9') { dot = 0;
if (cnt++ < 5) { for(;;) {
/* We simply ignore more than 5 digits precision. */ ch = *cp;
scale *= 10; if ( !dot && ch == '.' ) {
num = num * 10 + ch - '0'; scale = 1;
dot = 1;
} else if ( ch == '%' ) {
scale = dot ? scale*100 : 100;
cp++; /* % is always at the end */
break;
} else if ( ch >= '0' && ch <= '9' ) {
if ( scale < 100000 ) {
scale *= 10;
num = (num*10) + (ch-'0');
}
} else {
break;
} }
cp++; cp++;
} }
*cp_p = cp; *cp_p = cp;
/* special case: -M100 would mean 1.0 not 0.1 */
if (num == 100 && scale == 1000)
return MAX_SCORE;
/* user says num divided by scale and we say internally that /* user says num divided by scale and we say internally that
* is MAX_SCORE * num / scale. * is MAX_SCORE * num / scale.
*/ */
return (MAX_SCORE * num / scale); return (num >= scale) ? MAX_SCORE : (MAX_SCORE * num / scale);
} }
int diff_scoreopt_parse(const char *opt) int diff_scoreopt_parse(const char *opt)