From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Johan Claesson Newsgroups: gmane.emacs.bugs Subject: bug#23207: 25.1.50; file-notify event restarts the idle timer Date: Sun, 03 Apr 2016 15:22:05 +0200 Message-ID: <87d1q6kete.fsf@bredband.net> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1459689809 25450 80.91.229.3 (3 Apr 2016 13:23:29 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 3 Apr 2016 13:23:29 +0000 (UTC) To: 23207@debbugs.gnu.org Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sun Apr 03 15:23:16 2016 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1amhzn-0002Sx-63 for geb-bug-gnu-emacs@m.gmane.org; Sun, 03 Apr 2016 15:23:15 +0200 Original-Received: from localhost ([::1]:53480 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhzj-0006so-AH for geb-bug-gnu-emacs@m.gmane.org; Sun, 03 Apr 2016 09:23:11 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:50175) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhzd-0006se-NF for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:23:07 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1amhza-0000Br-Al for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:23:05 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:52383) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhza-0000Bn-76 for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:23:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1amhza-0004cL-2m for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:23:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Johan Claesson Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 03 Apr 2016 13:23:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 23207 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: X-Debbugs-Original-To: bug-gnu-emacs@gnu.org Original-Received: via spool by submit@debbugs.gnu.org id=B.145968974817710 (code B ref -1); Sun, 03 Apr 2016 13:23:01 +0000 Original-Received: (at submit) by debbugs.gnu.org; 3 Apr 2016 13:22:28 +0000 Original-Received: from localhost ([127.0.0.1]:49510 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1amhz2-0004bZ-96 for submit@debbugs.gnu.org; Sun, 03 Apr 2016 09:22:28 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:59167) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1amhz0-0004bL-1D for submit@debbugs.gnu.org; Sun, 03 Apr 2016 09:22:26 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1amhys-00008m-S8 for submit@debbugs.gnu.org; Sun, 03 Apr 2016 09:22:20 -0400 Original-Received: from lists.gnu.org ([2001:4830:134:3::11]:48448) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhys-00008i-OM for submit@debbugs.gnu.org; Sun, 03 Apr 2016 09:22:18 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:50132) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhyp-0006mo-IM for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:22:17 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1amhym-000082-6p for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:22:15 -0400 Original-Received: from smtprelay-h31.telenor.se ([213.150.131.4]:36019) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amhyl-00007t-LP for bug-gnu-emacs@gnu.org; Sun, 03 Apr 2016 09:22:12 -0400 Original-Received: from ipb4.telenor.se (ipb4.telenor.se [195.54.127.167]) by smtprelay-h31.telenor.se (Postfix) with ESMTP id AE44DC7E1 for ; Sun, 3 Apr 2016 15:22:06 +0200 (CEST) X-SENDER-IP: [213.114.146.248] X-LISTENER: [smtp.bredband.net] X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2DBCQDFFwFXPPiSctVWBxwBgxoeNS5PgnGDebQ3AQ1bgRcSB4cWORQBAQEBAQEBBgEBAQFCQIUeEwEnJA8lAQQZDApLiA0BmgGhEhmGB4hkBgEBBQNjhQgFjU2JcESPcI0nhhqJAB4BAYF4DD8DARiBTDowhk0BCBcDgRoBAQE X-IPAS-Result: A2DBCQDFFwFXPPiSctVWBxwBgxoeNS5PgnGDebQ3AQ1bgRcSB4cWORQBAQEBAQEBBgEBAQFCQIUeEwEnJA8lAQQZDApLiA0BmgGhEhmGB4hkBgEBBQNjhQgFjU2JcESPcI0nhhqJAB4BAYF4DD8DARiBTDowhk0BCBcDgRoBAQE X-IronPort-AV: E=Sophos;i="5.24,436,1454972400"; d="scan'208";a="270430269" Original-Received: from c-f89272d5.04-99-73746f3.cust.bredbandsbolaget.se (HELO goblin) ([213.114.146.248]) by ipb4.telenor.se with ESMTP; 03 Apr 2016 15:22:06 +0200 User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:115924 Archived-At: Hi, Every time there is a file-notify event the idle timer is restarted. This will postpone the triggering of idle timers. If there are periodical file-notify events they may prevent idle timers from ever triggering. The following recipe creates periodical file-notify events with the help of a normal non-idle timer. The poor idle timer started last in the recipe will never get to run. emacs -Q (with-current-buffer (find-file-noselect "/tmp/changing.txt") (setq auto-revert-interval 0.5) (auto-revert-mode)) (run-with-timer 1 1 (lambda () (write-region "x" nil "/tmp/changing.txt"))) (run-with-idle-timer 3 t (lambda () (message "Just idling"))) The same behavior is also present in 24.4 and 24.5 (i have not checked older versions). The idle timer gets restarted in read_char in keyboard.c. The following patch avoids this (but i am not sure it is the right thing to do). diff --git a/src/keyboard.c b/src/keyboard.c index 1a5dbd0..b71c656 100644 --- a/src/keyboard.c +++ b/src/keyboard.c @@ -2834,7 +2834,9 @@ read_char (int commandflag, Lisp_Object map, last_input_event = c; call4 (Qcommand_execute, tem, Qnil, Fvector (1, &last_input_event), Qt); - if (CONSP (c) && EQ (XCAR (c), Qselect_window) && !end_time) + if (CONSP (c) + && (EQ (XCAR (c), Qselect_window) || EQ (XCAR (c), Qfile_notify)) + && !end_time) /* We stopped being idle for this event; undo that. This prevents automatic window selection (under mouse_autoselect_window from acting as a real input event, for This is in the handing of special-event-map which includes file-notify. special-event-map includes some events that should restart the idle timer because they are user input (like drag-n-drop). But file-notify is not user input so i think it should not restart the idle timer. The same probably goes for dbus-event and config-changed-event that also are in the special-event-map. Maybe they also should be exempted in the same way. Also i don't understand why this code checks for Qselect_window. select-window is not included in special-event-map so that should never turn up there. Regards, /Johan In GNU Emacs 25.1.50.2 (x86_64-unknown-linux-gnu, X toolkit) of 2016-04-01 built on goblin Repository revision: 46edc38005ce47bdaae656a541858735a8ba5f26 Windowing system distributor 'The X.Org Foundation', version 11.0.11702000 System Description: Ubuntu 15.10 Configured using: 'configure --prefix=/ssd/home/jcl/usr --with-x-toolkit=lucid --without-toolkit-scroll-bars --disable-silent-rules --without-compress-install --enable-link-time-optimization PKG_CONFIG_PATH=/home/jcl/usr/lib/pkgconfig' Configured features: XAW3D XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GCONF GSETTINGS NOTIFY ACL LIBSELINUX GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB LUCID X11 Important settings: value of $LC_MONETARY: en_GB.UTF-8 value of $LC_NUMERIC: en_GB.UTF-8 value of $LC_TIME: en_GB.UTF-8 value of $LANG: en_US.UTF-8 value of $XMODIFIERS: @im=none locale-coding-system: utf-8-unix Major mode: Text Minor modes in effect: magit-auto-revert-mode: t display-time-mode: t minibuffer-depth-indicate-mode: t num3-mode: t minibuffer-electric-default-mode: t icomplete-mode: t diff-auto-refine-mode: t erc-list-mode: t erc-menu-mode: t erc-autojoin-mode: t erc-ring-mode: t erc-networks-mode: t erc-pcomplete-mode: t erc-track-mode: t erc-match-mode: t erc-button-mode: t erc-fill-mode: t erc-stamp-mode: t erc-netsplit-mode: t erc-irccontrols-mode: t erc-noncommands-mode: t erc-move-to-prompt-mode: t erc-readonly-mode: t global-git-commit-mode: t async-bytecomp-package-mode: t shell-dirtrack-mode: t wc-mode: t whitespace-mode: t winner-mode: t ido-everywhere: t which-function-mode: t global-eldoc-mode: t electric-layout-mode: t electric-indent-mode: t mouse-wheel-mode: t global-prettify-symbols-mode: t file-name-shadow-mode: t global-font-lock-mode: t font-lock-mode: t auto-composition-mode: t auto-encryption-mode: t auto-compression-mode: t temp-buffer-resize-mode: t line-number-mode: t auto-fill-function: do-auto-fill transient-mark-mode: t Recent messages: Replaced 0 occurrences Spell-checking region using aspell with default dictionary...done Spell-checking suspended; use C-u M-$ to resume Saving file /ssd/home/jcl/IDLE_TIMER_REPORT.el... Wrote /ssd/home/jcl/IDLE_TIMER_REPORT.el Saving file /ssd/home/jcl/IDLE_TIMER_REPORT.el... Wrote /ssd/home/jcl/IDLE_TIMER_REPORT.el Mark set Saving file /ssd/home/jcl/IDLE_TIMER_REPORT.el... Wrote /ssd/home/jcl/IDLE_TIMER_REPORT.el Load-path shadows: None found. Features: (nnir emacsbug nndoc gnus-dup esh-var esh-io esh-cmd esh-opt esh-ext esh-proc esh-arg esh-groups eshell esh-module esh-mode esh-util tree-widget inferior-slime slime-banner slime-cl-indent cl-indent elp gud tar-mode edmacro let-alist color calc-bin calc-ext flow-fill shadow semantic/symref/grep semantic/symref semantic/util-modes semantic/util semantic semantic/tag semantic/lex semantic/fw mode-local cedet vc-bzr vc-src vc-sccs vc-svn vc-cvs vc-rcs tabify gnus-dired tramp-cache hippie-exp bug-reference w3m-form gnus-cite mail-extr gnus-async pulse w3m-cookie smiley jcl-replace jcl-games-init jcl-muse-init htmlize-hack htmlize muse-latex muse-html muse-xml-common muse-colors muse-publish muse-project muse-protocols muse-regexps muse muse-nested-tags muse-mode jcl-yaoddmuse-init yaoddmuse-extension yaoddmuse skeleton sgml-mode eieio-opt speedbar sb-image ezimage dframe wiki-summary autoload lisp-mnt cus-edit cus-start cus-load time-stamp misearch multi-isearch jcl-renegade-goblin jcl-home-boot vc vc-dispatcher vc-git org-rmail org-mhe org-irc org-info org-gnus org-docview org-bibtex bibtex org-bbdb org-w3m jcl-load jcl-billboard-init typing-practice jcl-elisp-init profiler tagbag-init jcl-grep-init jcl-abbrev-init jcl-ediff-init jcl-term-init jcl-erlang-init jcl-sbg jcl-erlang-log bookmark jcl-erlang-man man jcl-midnite-init jcl-midnite midnight jcl-picpocket-init picpocket-test jcl-erc-init jcl-ido-init jcl-dired-init image-file wdired jcl-register-init jcl-command-subset-init jcl-generic-init generic-x jcl-eww-init eww url-queue shr svg dom jcl-org-init poporg ob-latex ox-beamer org-feed org-capture jcl-shell-init jcl-haskell-init jcl-haskell-old inf-haskell haskell-decl-scan haskell-mode haskell-cabal haskell-utils haskell-font-lock haskell-indentation haskell-string haskell-sort-imports haskell-lexeme haskell-align-imports haskell-compat haskell-complete-module flymake dabbrev haskell-customize jcl-mentor-init mentor xml-rpc url-scgi jcl-google-init jcl-modes-init checkdoc magit-blame magit-stash magit-bisect magit-remote magit-commit magit-sequence magit magit-apply magit-wip magit-log magit-diff smerge-mode magit-core magit-autorevert magit-process magit-popup magit-mode magit-git crm magit-section magit-utils calc calc-loaddefs calc-macs jcl-rfc rfcview goto-addr proced nroff-mode table picture python rx jcl-ruby inf-ruby ruby-mode fpl macrostep-c cmacexp cwarn grep jcl-elpa-init ffap view jcl-global-init time printing mb-depth saveplace num3-mode minibuf-eldef icomplete jcl-sl-init jcl-keys-init kmacro diff-mode jcl-sbg-ssit jcl-google google-translate google-translate-default-ui google-translate-core-ui google-translate-core google-translate-tk jcl-imenu jcl-wesnoth wesnoth-mode wesnoth-wml-data wesnoth-update jcl-erlang jcl-sbg-compile jcl-picpocket jcl-goblin jcl-dired jcl-dired-lexical picpocket fuzzy-match jcl-ediff jcl-elisp jcl-grep edebug jcl-stumpwm jcl-org jcl-erc erc-list erc-menu erc-join erc-ring erc-networks erc-pcomplete erc-track erc-match erc-button erc-fill erc-stamp erc-netsplit erc-goodies erc erc-backend erc-compat jcl-vbox jcl-lxc jcl-keys windmove jcl-debbugs-init debbugs-org debbugs-gnu debbugs soap-client warnings rng-xsd rng-dt rng-util xsd-regexp jcl-isearch jcl-sbg-scan tagbag sbg filecache jcl-bugfix loadhist git-commit help-fns with-editor async-bytecomp async tramp-sh tramp tramp-compat tramp-loaddefs trampver ucs-normalize shell log-edit pcvs-util add-log dash apropos jcl-merge jcl-wip jcl-work-buffer ediff-merg ediff-wind ediff-diff ediff-mult ediff-help ediff-init ediff-util ediff jcl-sty jcl-epresent face-remap org-indent epresent ox-latex ox-icalendar ox-html ox-ascii ox-publish ox org-element avl-tree diet ps-print ps-print-loaddefs ps-def lpr jcl-template jcl-copyright jcl-template-lib cperl-mode cc-mode cc-fonts cc-guess cc-menus cc-cmds cc-styles cc-align cc-engine cc-vars cc-defs erlang tempo wc-mode jcl-register jcl-advice jcl-command-subset jcl-duff clearcase reporter jcl-motion jcl-safe-init jcl-site-start jpt-mode erlang-start qp gnus-ml gnus-topic cursor-sensor mm-archive network-stream nsm url-http url-gw puny url-cache url-auth nnrss mm-url nndraft nnmh nnfolder nnml jcl-master jcl-calendar-init jcl-emms-init emms-playlist-limit emms-volume emms-volume-amixer emms-i18n emms-history emms-score emms-stream-info emms-metaplaylist-mode emms-bookmarks emms-cue emms-mode-line-icon emms-browser sort emms-playlist-sort emms-last-played emms-player-xine emms-player-mpd tq emms-streams emms-show-all emms-tag-editor emms-mark emms-cache emms-info-ogginfo emms-info-mp3info emms-player-vlc emms-player-mplayer jcl-emms emms-mode-line emms-librefm-stream emms-librefm-scrobbler emms-playing-time emms-info later-do emms-playlist-mode emms-source-playlist xml emms-setup emms-lyrics emms-url url url-proxy url-privacy url-expand url-methods url-history url-cookie url-domsuf emms-source-file locate emms-player-simple emms emms-compat jcl-gnus-init jcl-gnus jcl-w3m-init jcl-term ange-ftp w3m-search w3m-bookmark w3m-antenna w3m-rss jcl-org-import-icalendar jcl-status jcl-buffer-init org-location-google-maps org-agenda google-maps google-maps-static url-util google-maps-geocode google-maps-base json map org org-macro org-footnote org-pcomplete pcomplete org-list org-faces org-entities org-version ob-emacs-lisp ob ob-tangle ob-ref ob-lob ob-table ob-exp org-src ob-keys ob-comint ob-core ob-eval org-compat org-macs org-loaddefs autorevert filenotify appt jcl-calendar jcl-swedish-postfix quail holidays hol-loaddefs icalendar diary-lib diary-loaddefs cal-menu calendar cal-loaddefs w3m doc-view jka-compr image-mode timezone w3m-hist w3m-fb bookmark-w3m w3m-ems w3m-ccl ccl w3m-favicon w3m-image w3m-proc w3m-util dired-aux dired-x gnus-agent gnus-srvr gnus-score score-mode nnvirtual nntp gnus-cache gnus-msg gnus-art mm-uu mml2015 mm-view mml-smime smime dig gnus-sum gnus-group gnus-undo gnus-start gnus-cloud nnimap nnmail mail-source tls gnutls utf7 netrc nnoo parse-time gnus-spec gnus-int gnus-range gnus-win mailcap starttls smtpmail sendmail message format-spec rfc822 mml mml-sec epa derived epg mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader gnus nnheader gnus-util rmail rmail-loaddefs rfc2047 rfc2045 ietf-drums mail-utils mm-util mail-prsvr wid-edit jcl-boot jcl-slime-init slime-fancy slime-trace-dialog slime-fontifying-fu slime-package-fu slime-references slime-compiler-notes-tree slime-scratch slime-presentations bridge slime-macrostep macrostep slime-mdot-fu slime-enclosing-context slime-fuzzy slime-fancy-trace slime-fancy-inspector slime-c-p-c slime-autodoc jcl-misc jcl-ert ispell find-lisp jcl-misc-lexical whitespace winner jcl-windows jcl-buffer ido ert find-func ewoc debug jcl-compile slime-editing-commands slime-repl slime-parse slime compile etags xref project arc-mode archive-mode noutline outline easy-mmode pp comint ansi-color hyperspec thingatpt browse-url jcl-face hi-lock dired dired-loaddefs paren mic-paren which-func imenu hl-line jcl-times-init jcl-early-init jcl-compat-init advice jcl-util seq subr-x jcl-times server term disp-table ehelp ring sh-script smie executable jcl-load-path tex-site eimp-autoloads gnugo-autoloads ascii-art-to-unicode-autoloads htmlize-autoloads ipython-autoloads list-utils-autoloads cl memory-usage-autoloads muse-autoloads num3-mode-autoloads finder-inf python-autoloads slime-autoloads info package epg-config url-handlers url-parse auth-source cl-seq eieio byte-opt bytecomp byte-compile cl-extra help-mode easymenu cconv eieio-core cl-macs gv eieio-loaddefs cl-loaddefs pcase cl-lib password-cache url-vars time-date mule-util tooltip eldoc electric uniquify ediff-hook vc-hooks lisp-float-type mwheel term/x-win x-win term/common-win x-dnd tool-bar dnd fontset image regexp-opt fringe tabulated-list newcomment elisp-mode lisp-mode prog-mode register page menu-bar rfn-eshadow timer select scroll-bar mouse jit-lock font-lock syntax facemenu font-core term/tty-colors frame cl-generic cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech european ethiopic indian cyrillic chinese charscript case-table epa-hook jka-cmpr-hook help simple abbrev obarray minibuffer cl-preloaded nadvice loaddefs button faces cus-face macroexp files text-properties overlay sha1 md5 base64 format env code-pages mule custom widget hashtable-print-readable backquote dbusbind inotify dynamic-setting system-font-setting font-render-setting x-toolkit x multi-tty make-network-process emacs) Memory information: ((conses 16 2659061 392210) (symbols 48 102906 0) (miscs 40 6241 8580) (strings 32 502665 71858) (string-bytes 1 17223972) (vectors 16 136297) (vector-slots 8 2761520 106348) (floats 8 1490 1629) (intervals 56 317146 3703) (buffers 976 171) (heap 1024 216403 18278))