Steffen Prohaska <prohaska@xxxxxx> writes: > From: Dmitry Kakurin <Dmitry.Kakurin@xxxxxxxxx> > > Signed-off-by: Dmitry Kakurin <Dmitry.Kakurin@xxxxxxxxx> > Signed-off-by: Steffen Prohaska <prohaska@xxxxxx> > --- > convert.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/convert.c b/convert.c > index 1c66844..f24ac25 100644 > --- a/convert.c > +++ b/convert.c > @@ -61,6 +61,10 @@ static void gather_stats(const char *buf, unsigned long size, struct text_stat * > else > stats->printable++; > } > + > + // If file ends with EOF then don't count this EOF as non-printable > + if ( size >= 1 && buf[size-1] == '\032' ) > + stats->nonprintable--; Style. I debated for 5 seconds with myself if this should be inside #ifdef, but doing this everywhere would give us reproducibility --- otherwise the resulting project won't be cross platform, so I think the intention of this change is good. -- 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
