Subject: bug#25445: Bug (Trunk): Redirect fails in Duckduckgo

From: Eli Zaretskii <[email-masked]>
Date: Mon, 16 Jan 2017 21:24:35 +0200

> From: Noam Postavsky <[email-masked]>
> Date: Mon, 16 Jan 2017 13:24:08 -0500
> Cc: [email-masked]
> 
> On Mon, Jan 16, 2017 at 1:12 PM, Achim Gratz <[email-masked]> wrote:
> > [email-masked] writes:
> >>        (when (or (string-match "^\\([0-9]+\\) *;.*url=\"\\([^\"]+\\)\"" refresh)
> >> +                (string-match "^\\([0-9]+\\) *;.*url=\'\\([^\']+\\)\'" refresh)
> >>                  (string-match "^\\([0-9]+\\) *;.*url=\\([^ ]+\\)" refresh))
> >
> > Why not use a regex with alternatives and do just a single match?
> 
> Stefan made the same suggestion:
> https://lists.gnu.org/archive/html/emacs-devel/2017-01/msg00268.html
> 
> I'm not entirely sure it's an improvement.

Neither am I: it would make the regexp harder to interpret (for
humans).