From: Matthieu Moy Date: Wed, 17 Jun 2015 14:54:04 GMT Subject: Re: [PATCH/RFC v4 10/10] send-email: suppress meaningless whitespaces in from field Message-ID: In-Reply-To: <1434551525-24709-1-git-send-email-remi.lespinet@ensimag.grenoble-inp.fr> Remi Lespinet writes: > if (defined $sender) { > + $sender =~ s/^\s+|\s$//g; I would say \s+ also for the second \s. Not really different, but it feels wrong to iterate the substitution as many times as there are trailing spaces to remove. -- Matthieu Moy http://www-verimag.imag.fr/~moy/