Commits

Anonymous committed cf21919

Fix up progress report for off-by-one error

We used to print the index of the object we unpacked, not how many we
had unpacked. Which caused slightly confusing progress reports like

100% (2/3) done

rather than the more obvious "3/3" for 100% ;)

  • Participants
  • Parent commits 4bc5fbf

Comments (0)

Files changed (1)

 		static unsigned long last_sec;
 		static unsigned last_percent;
 		struct timeval now;
-		unsigned percentage = ((1+nr) * 100) / total;
+		unsigned percentage = (nr * 100) / total;
 
 		gettimeofday(&now, NULL);
 		if (percentage != last_percent || now.tv_sec != last_sec) {
 
 	use(sizeof(struct pack_header));
 	for (i = 0; i < nr_objects; i++)
-		unpack_one(i, nr_objects);
+		unpack_one(i+1, nr_objects);
 	if (delta_list)
 		die("unresolved deltas left after unpacking");
 }