You mean you expect me to learn how to use emacs lisp correctly 
rather than use ugly hacks.... ?

Fine.  How about adding a (raise-frame) then ?



----- Mail original -----
De: "Tassilo Horn" <t...@gnu.org>
À: can...@free.fr
Cc: 702...@bugs.debian.org, bug-auc...@gnu.org
Envoyé: Vendredi 12 Avril 2013 18:52:18
Objet: Re: Bug#702871: Regarding forward and backward search with evince.

can...@free.fr writes:

Hi!

> Let me add some info to my original bug report, regarding forward
> search with evince.  In fact, some odd behaviour persisted (with or
> without my patch), in particular evince remained first in the Alt-Tab
> application list, regardless of swithching between other applications.
> While I couldn't find a reference for the evince DBus API, I decided
> to check what would happen if I replace the timestamp argument with a
> constant zero (or one), and it solved everything (including raising
> and giving focus).

The broken timestamp value was the source of another problem, too.  This
has already been solved by providing 0 as timestamp, so the current Git
version of AUCTeX already contains the fix.

> Can anyone tell me what this parameter even mean?

0 stands for NOW.

> Anyway, raising the emacs window via wmctrl on reverse search, as in
> the previous patch, and replacing the complicated calculation of
> timestamp with some constant in forward search, seems to give a fully
> correct behaviour.

The wmctrl thingy is not acceptable, sorry.

Bye,
Tassilo
Description: <short summary of the patch>
 TODO: Put a short summary on the line above and replace this paragraph
 with a longer explanation of this change. Complete the meta-information
 with other relevant fields (see below for details). To make it easier, the
 information below has been extracted from the changelog. Adjust it or drop
 it.
 .
 auctex (11.87-1.3) unstable; urgency=low
 .
   * Raise windows on reverse search.
   * Set evince forward search timestamp to zero.
Author: Itaï BEN YAACOV <can...@free.fr>

---
The information above should follow the Patch Tagging Guidelines, please
checkout http://dep.debian.net/deps/dep3/ to learn about the format. Here
are templates for supplementary fields that you might want to add:

Origin: <vendor|upstream|other>, <url of original patch>
Bug: <url in upstream bugtracker>
Bug-Debian: http://bugs.debian.org/<bugnumber>
Bug-Ubuntu: https://launchpad.net/bugs/<bugnumber>
Forwarded: <no|not-needed|url proving that it has been forwarded>
Reviewed-By: <name and email of someone who approved the patch>
Last-Update: <YYYY-MM-DD>

--- auctex-11.87.orig/tex.el
+++ auctex-11.87/tex.el
@@ -1508,7 +1508,8 @@ or newer."
       (goto-char (point-min))
       (forward-line (1- line))
       (unless (= col -1)
-        (move-to-column col)))))
+        (move-to-column col))
+      (raise-frame))))
 
 (define-minor-mode TeX-source-correlate-mode
   "Minor mode for forward and inverse search.

Reply via email to