Re: [hackers] [sbase][patch] find: empty line means no for -ok

2015-06-18 Thread Roberto E. Vargas Caballero
Hi, > Yes I most certainly did, this is what I get for submitting patches > without testing. The shame. New patch attached, also protects against > the glibc bug causing fgetc to hang after EOF was received. And what about if we send a patch to glibc instead? Regards,

Re: [hackers] [sbase][patch] find: empty line means no for -ok

2015-06-18 Thread Evan Gates
On Thu, Jun 18, 2015 at 3:57 PM, Wolfgang Corcoran-Mathe wrote: > emg, > > Quoth Evan Gates on Thu, Jun 18 2015 14:47 -0700: >> >> diff --git a/find.c b/find.c >> index 186263b..0de1951 100644 >> --- a/find.c >> +++ b/find.c >> @@ -415,10 +415,9 @@ pri_ok(struct arg *arg) >> reply = fgetc(

Re: [hackers] [sbase][patch] find: empty line means no for -ok

2015-06-18 Thread Wolfgang Corcoran-Mathe
emg, Quoth Evan Gates on Thu, Jun 18 2015 14:47 -0700: diff --git a/find.c b/find.c index 186263b..0de1951 100644 --- a/find.c +++ b/find.c @@ -415,10 +415,9 @@ pri_ok(struct arg *arg) reply = fgetc(stdin); /* throw away rest of line */ - while ((c = fgetc(stdin)) != '\n'

[hackers] [sbase][patch] find: empty line means no for -ok

2015-06-18 Thread Evan Gates
As Wolfgang pointed out if the user hit return at the prompt for -ok another return was needed to continue. This patch fixes that accepting an empty line as no. -emg From 65656d9b2aaf63c518a0e461c373bb7392b5653e Mon Sep 17 00:00:00 2001 From: Evan Gates Date: Thu, 18 Jun 2015 14:43:11 -0700 Subje