* [PATCH] show_stats(): fix stats width calculation
@ 2008-06-27 0:18 Olivier Marin
0 siblings, 0 replies; only message in thread
From: Olivier Marin @ 2008-06-27 0:18 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano
From: Olivier Marin <dkr@freesurf.fr>
Before this patch, name_width becomes negative or null for width values
less than 15 and name_width values greater than 25 (default: 50). This
leads to output random data.
This patch checks for minimal width and name_width values.
Signed-off-by: Olivier Marin <dkr@freesurf.fr>
---
diff.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/diff.c b/diff.c
index 44e8790..a85c8ca 100644
--- a/diff.c
+++ b/diff.c
@@ -830,12 +830,12 @@ static void show_stats(struct diffstat_t* data, struct diff_options *options)
/* Sanity: give at least 5 columns to the graph,
* but leave at least 10 columns for the name.
*/
- if (width < name_width + 15) {
- if (name_width <= 25)
- width = name_width + 15;
- else
- name_width = width - 15;
- }
+ if (width < 25)
+ width = 25;
+ if (name_width < 10)
+ name_width = 10;
+ else if (width < name_width + 15)
+ name_width = width - 15;
/* Find the longest filename and max number of changes */
reset = diff_get_color_opt(options, DIFF_RESET);
--
1.5.6.1.141.ge3ab.dirty
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2008-06-27 0:19 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-27 0:18 [PATCH] show_stats(): fix stats width calculation Olivier Marin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox