Re: [Bug-AUCTeX] tex-source-correlate-function and %(outpage) expander

[Ralf Angeli]

Please keep the mailing list copied!  And please don't send HTML mails.

* Vilmos Prokaj (2011-05-15) writes:

> Thanks your quick answer. I checked my version which comes from the 
> latest Ubuntu, so I did not expect that a more than 3 years old version 
> is included. I also checked the latest version on the net.
>
> In both cases in tex.el I found
>
>    ("%(outpage)" (lambda ()
>              (if TeX-source-correlate-output-page-function
>              (funcall TeX-source-correlate-output-page-function)
>                "1")))
>
> My suggestion was to add an "or" function to handle the case when 
> TeX-source-... function is defined but its return value is nil. It 
> happens when the output of synctex is just a version string.
>
> The proposed modification is small indicated with red
>
> ("%(outpage)" (lambda ()
>       *(or*   
>       (if TeX-source-correlate-output-page-function
>                 (funcall TeX-source-correlate-output-page-function)*)*;; end 
> of if
>             "1")))

Sorry, I missed the `or' in your first mail.  I agree that it's a good
idea to remove the burden from the output page functions to always
return a number, so I've changed the expander function accordingly.
I've used `when' instead of `if' which makes it clear that there is no
`else' part.

In any case, the change was not really necessary for the current set of
output page functions because they all return a number.  I've just made
it so that there will be no problems with functions which might be added
in the future.

-- 
Ralf