Re: [Nicolas Dudebout] Re: [Bug-AUCTeX] 11.84; Filename containing spaces in Mac OS X

[David Kastrup]

Ralf Angeli <address@hidden> writes:

> * Ralf Angeli (2007-08-07) writes:
>
>> I can reproduce the bug with the View command not being shown with a
>> file called "space test.tex" as simple as
>>
>> \documentclass{article}
>> \begin{document}
>> foo
>> \end{document}
>
> The cause is that `TeX-output-extension' is set to something like
> "dvi\"" by `TeX-TeX-sentinel-check'.  The following patch should fix
> this.  Does anybody see a more elegant way?
>
> --- tex-buf.el        10 Jul 2007 22:58:43 +0200      1.261
> +++ tex-buf.el        07 Aug 2007 23:19:53 +0200      
> @@ -836,7 +836,7 @@
>  Return nil ifs no errors were found."
>    (save-excursion
>      (goto-char (point-max))
> -    (if (re-search-backward "^Output written on \\(.*\\) (\\([0-9]+\\) page"
> +    (if (re-search-backward "^Output written on \"?\\([^\"]*\\)\"? 
> (\\([0-9]+\\) page"
>                           nil t)

That is going to be dead slow.  First, "." is equivalent to "[^\n]",
and you don't exclude \n.  Then both quotes are optional, so the *
tries creeping out over everything (a problem with the previous code,
too).  One should use *? here so that the smallest match wins.
Anyway, I'd try something like
"\\(?:\"\\([^\n\\\"]+?\\)\"\\|\\([^\n\\\\" ]\\)\\) (\\([0-9+]\\) page"
or so, adjusting to (or (match-string 1) (match-string 2)) where
necessary.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum