Subject: Re: bug#24317: 25.1.50; image-dired-rotate-original: regexp doesn't match .jpeg suffix

From: Eli Zaretskii <[email-masked]>
Date: Sat, 27 Aug 2016 11:57:47 +0300

> From: Tino Calancha <[email-masked]>
> Date: Sat, 27 Aug 2016 17:53:49 +0900 (JST)
> cc: Tino Calancha <[email-masked]>, [email-masked]
> 
> > I understand why we need to fix a typo in the regexp (which the log
> > entry doesn't even mention, btw), but what's the rationale for making
> > it a defvar?
> I made a defvar because it was used in 2 functions: only one show the 
> typo.

In that case, why not make a function that does the string-match as
well, in which case you won't need a variable, because it will appear
in a single place, and will also have the test localized to a single
function?

Thanks.