Re: md5: convert from fgetln(3) to getline(3)

2018-09-05 Thread Todd C. Miller
On Thu, 23 Aug 2018 11:24:54 -0500, Scott Cheloha wrote: > On Tue, Aug 14, 2018 at 03:11:47PM -0500, Scott Cheloha wrote: > > This patch is ok cheloha@ and I can commit if someone else > > is ok, too. > > 1 week bump. Anyone else ok? OK millert@ - todd

Re: md5: convert from fgetln(3) to getline(3)

2018-09-05 Thread Lauri Tirkkonen
On Thu, Aug 23 2018 11:24:54 -0500, Scott Cheloha wrote: > On Tue, Aug 14, 2018 at 03:11:47PM -0500, Scott Cheloha wrote: > > This patch is ok cheloha@ and I can commit if someone else > > is ok, too. > > 1 week bump. Anyone else ok? no takers? -- Lauri Tirkkonen | lotheac @ IRCnet

Re: md5: convert from fgetln(3) to getline(3)

2018-08-23 Thread Scott Cheloha
On Tue, Aug 14, 2018 at 03:11:47PM -0500, Scott Cheloha wrote: > This patch is ok cheloha@ and I can commit if someone else > is ok, too. 1 week bump. Anyone else ok? > On Tue, Aug 14, 2018 at 10:47:07PM +0300, Lauri Tirkkonen wrote: > > [...] > > > > diff --git a/bin/md5/md5.c b/bin/md5/md5.c

Re: md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Scott Cheloha
This patch is ok cheloha@ and I can commit if someone else is ok, too. On Tue, Aug 14, 2018 at 10:47:07PM +0300, Lauri Tirkkonen wrote: > [...] > > diff --git a/bin/md5/md5.c b/bin/md5/md5.c > index 84adbcf0989..1abf28cfa16 100644 > --- a/bin/md5/md5.c > +++ b/bin/md5/md5.c > @@ -549,11 +549,11 @

Re: md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Lauri Tirkkonen
On Tue, Aug 14 2018 14:29:30 -0500, Scott Cheloha wrote: > On Tue, Aug 14, 2018 at 08:55:14PM +0300, Lauri Tirkkonen wrote: > > Hi, > > > > On Tue, Aug 14 2018 12:08:54 -0500, Scott Cheloha wrote: > > > > + len = (size_t)linelen; > > > > + if (line[len - 1] == '\n') > >

Re: md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Scott Cheloha
On Tue, Aug 14, 2018 at 08:55:14PM +0300, Lauri Tirkkonen wrote: > Hi, > > On Tue, Aug 14 2018 12:08:54 -0500, Scott Cheloha wrote: > > > + len = (size_t)linelen; > > > + if (line[len - 1] == '\n') > > > + line[len - 1] = '\0'; > > > > You can just use linelen dire

Re: md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Lauri Tirkkonen
Hi, On Tue, Aug 14 2018 12:08:54 -0500, Scott Cheloha wrote: > > + len = (size_t)linelen; > > + if (line[len - 1] == '\n') > > + line[len - 1] = '\0'; > > You can just use linelen directly and skip the assignment. I did it this way because this bit later

Re: md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Scott Cheloha
On Tue, Aug 14, 2018 at 03:34:53PM +0300, Lauri Tirkkonen wrote: > Similar to previous diffs for join(1) and paste(1). No intended > functional change. I agree with the direction. Comments inline. > diff --git a/bin/md5/md5.c b/bin/md5/md5.c > index 84adbcf0989..a8f18262a92 100644 > --- a/bin/md

md5: convert from fgetln(3) to getline(3)

2018-08-14 Thread Lauri Tirkkonen
Similar to previous diffs for join(1) and paste(1). No intended functional change. diff --git a/bin/md5/md5.c b/bin/md5/md5.c index 84adbcf0989..a8f18262a92 100644 --- a/bin/md5/md5.c +++ b/bin/md5/md5.c @@ -549,11 +549,11 @@ digest_filelist(const char *file, struct hash_function *defhash, int se