Subject: Re: bug#24057: 25.1.50; ffap interprets comments beginning with "//" as file path

From: Eli Zaretskii <[email-masked]>
Date: Tue, 26 Jul 2016 17:41:33 +0300

> From: Kaushal Modi <[email-masked]>
> Date: Mon, 25 Jul 2016 20:18:27 +0000
> Cc: [email-masked], [email-masked]
> 
> Here a git formatted patch .. it's an optimized, generic version of the
> snippet in my previous email. It's now "/" agnostic.

Thanks.  Most of the patch looks like whitespace changes?

> +        (setq beg (catch 'break
> +                    (while (< beg-new end)
> +                      (goto-char beg-new)
> +                      (if (nth 4 (syntax-ppss)) ; in a comment
> +                          (throw 'break beg-new)
> +                        (setq beg-new (1+ beg-new))))
> +                    end)))) ; Set BEG to END if no throw happens

Is there any problem with a more conventional while loop that stops
when syntax-ppss has its 5th element non-nil?  Why did you need to use
throw and catch here?