Re: [PATCH 0/6] fix various minor issues and warnings

[Ruediger Meier <sweet_f_a@gmx.de>]

On Wednesday 14 June 2017, Karel Zak wrote:
> On Tue, Jun 13, 2017 at 05:18:33PM +0200, R=FCdiger Meier wrote:
> > On 06/13/2017 03:47 PM, Karel Zak wrote:
> > > On Mon, Jun 12, 2017 at 10:49:52AM +0200, Ruediger Meier wrote:
> > > > From: Ruediger Meier <ruediger.meier@ga-group.nl>
> > > >
> > > > Ruediger Meier (6):
> > > >    tests: fix fincore, don't use variable COLUMNS
> > > >    libmount: btrfs, remove unused setter functions
> > > >    hwclock: fix warning [-Winvalid-noreturn]
> > > >    misc: fix gcc-7 sprintf warnings -Wformat-overflow
> > > >    misc: fix gcc-7 snprintf warnings -Wformat-truncation
> > > >    libblkid: fix gcc-7 warning -Wint-in-bool-context
> > > >
> > > >   lib/loopdev.c                    |  2 +-
> > > >   lib/sysfs.c                      |  4 ++--
> > > >   libblkid/src/devname.c           |  2 +-
> > > >   libblkid/src/superblocks/exfat.c |  4 ++--
> > > >   libmount/src/btrfs.c             |  4 ----
> > > >   login-utils/last.c               |  2 +-
> > > >   sys-utils/hwclock.c              |  2 +-
> > > >   sys-utils/hwclock.h              |  3 ++-
> > > >   tests/ts/fincore/count           | 12 ++++++------
> > > >   9 files changed, 16 insertions(+), 19 deletions(-)
> > >
> > > Seems better than the previous version from Sami (May 27).
> >
> > These fixed are completely independent/unrelated to the other pull
> > request from Sami. Sami's PR just motivated me to try out gcc-7
> > too.
>
> Yep, you both tried to fix the sprintf warnings, your version with
> NAME_MAX seems better.

Ooops, I didn't even noticed and could merge all together without=20
confilcts.

> Applied, thanks.
>
>     Karel