]> code.delx.au - gnu-emacs-elpa/blobdiff - packages/ggtags/ggtags.el
Merge branch 'master' of github.com:leoliu/ggtags
[gnu-emacs-elpa] / packages / ggtags / ggtags.el
index 32a069eed1fd8c617d2cea10e79695a1a55bd530..541025a22bac735c7ba61084dc80da19cea984a8 100644 (file)
@@ -3,11 +3,11 @@
 ;; Copyright (C) 2013-2014  Free Software Foundation, Inc.
 
 ;; Author: Leo Liu <sdl.web@gmail.com>
-;; Version: 0.7.12
+;; Version: 0.8.0
 ;; Keywords: tools, convenience
 ;; Created: 2013-01-29
 ;; URL: https://github.com/leoliu/ggtags
-;; Package-Requires: ((emacs "24"))
+;; Package-Requires: ((emacs "24") (cl-lib "0.5"))
 
 ;; This program is free software; you can redistribute it and/or modify
 ;; it under the terms of the GNU General Public License as published by
 ;;
 ;; Usage:
 ;;
-;; Type `M-x ggtags-mode' to enable the minor mode, or as usual enable
-;; it in your desired major mode hooks. When the mode is on the symbol
-;; at point is underlined if it is a valid (definition) tag.
+;; `ggtags' is similar to the standard `etags' package. These keys
+;; `M-.', `M-,', `M-*' and `C-M-.' should work as expected in
+;; `ggtags-mode'. See the README in https://github.com/leoliu/ggtags
+;; for more details.
 ;;
-;; `M-.' finds definition or references according to the context at
-;; point, i.e. if point is at a definition tag find references and
-;; vice versa. `M-]' finds references.
-;;
-;; If multiple matches are found, navigation mode is entered, the
-;; mode-line lighter changed, and a navigation menu-bar entry
-;; presented. In this mode, `M-n' and `M-p' moves to next and previous
-;; match, `M-}' and `M-{' to next and previous file respectively.
-;; `M-o' toggles between full and abbreviated displays of file names
-;; in the auxiliary popup window. When you locate the right match,
-;; press RET to finish which hides the auxiliary window and exits
-;; navigation mode. You can continue the search using `M-,'. To abort
-;; the search press `M-*'.
-;;
-;; Normally after a few searches a dozen buffers are created visiting
-;; files tracked by GNU Global. `C-c M-k' helps clean them up.
-;;
-;; Check the menu-bar entry `Ggtags' for other useful commands.
+;; All commands are made available in the menu-bar entry `Ggtags' in
+;; `ggtags-mode'.
 
 ;;; Code:
 
 (eval-when-compile
-  (require 'cl)
   (require 'url-parse))
 
+(require 'cl-lib)
+(require 'ewoc)
 (require 'compile)
 (require 'etags)
 (require 'tabulated-list)               ;preloaded since 24.3
       (list 'progn (list 'defvar var val docstring)
             (list 'make-variable-buffer-local (list 'quote var)))))
 
-  (defmacro* when-let ((var exp) &rest body)
-    "A macro that combines `let' and `when'."
-    (declare (indent 1) (debug ((sexp form) body)))
-    `(let ((,var ,exp)) (when ,var ,@body))))
+  (defmacro ignore-errors-unless-debug (&rest body)
+    "Ignore all errors while executing BODY unless debug is on."
+    (declare (debug t) (indent 0))
+    `(condition-case-unless-debug nil (progn ,@body) (error nil))))
 
 (eval-and-compile
-  (or (fboundp 'user-error)
-      (defalias 'user-error 'error)))
+  (or (fboundp 'user-error)             ;24.3
+      (defalias 'user-error 'error))
+  (or (fboundp 'read-only-mode)         ;24.3
+      (defalias 'read-only-mode 'toggle-read-only))
+  (or (fboundp 'register-read-with-preview) ;24.4
+      (defalias 'register-read-with-preview 'read-char)))
 
 (defgroup ggtags nil
   "GNU Global source code tagging system."
   "Face used to highlight matched line in Global buffer."
   :group 'ggtags)
 
+(defcustom ggtags-executable-directory nil
+  "If non-nil the directory to search global executables."
+  :type '(choice (const :tag "Unset" nil) directory)
+  :risky t
+  :group 'ggtags)
+
 (defcustom ggtags-oversize-limit (* 10 1024 1024)
   "The over size limit for the  GTAGS file.
 For large source trees, running 'global -u' can be expensive.
@@ -105,6 +101,29 @@ automatically switches to 'global --single-update'."
                  number)
   :group 'ggtags)
 
+(defcustom ggtags-global-always-update nil
+  "If non-nil always update tags for current file on save."
+  :safe 'booleanp
+  :type 'boolean
+  :group 'ggtags)
+
+(defcustom ggtags-include-pattern
+  '("^\\s-*#\\(?:include\\|import\\)\\s-*[\"<]\\(?:[./]*\\)?\\(.*?\\)[\">]" . 1)
+  "Pattern used to detect #include files.
+Value can be (REGEXP . SUB) or a function with no arguments."
+  :type '(choice (const :tag "Disable" nil)
+                 (cons regexp integer)
+                 function)
+  :safe 'stringp
+  :group 'ggtags)
+
+(defcustom ggtags-use-project-gtagsconf t
+  "Non-nil to use GTAGSCONF file found at project root.
+File .globalrc and gtags.conf are checked in order."
+  :safe 'booleanp
+  :type 'boolean
+  :group 'ggtags)
+
 (defcustom ggtags-project-duration 600
   "Seconds to keep information of a project in memory."
   :type 'number
@@ -128,12 +147,12 @@ directory local variables is not enabled by default per
   :group 'ggtags)
 
 (defcustom ggtags-global-window-height 8 ; ggtags-global-mode
-  "Number of lines for the 'global' popup window.
+  "Number of lines for the *ggtags-global* popup window.
 If nil, use Emacs default."
   :type '(choice (const :tag "Default" nil) integer)
   :group 'ggtags)
 
-(defcustom ggtags-global-abbreviate-filename 35
+(defcustom ggtags-global-abbreviate-filename 40
   "Non-nil to display file names abbreviated e.g. \"/u/b/env\".
 If an integer abbreviate only names longer than that number."
   :type '(choice (const :tag "No" nil)
@@ -152,7 +171,7 @@ If an integer abbreviate only names longer than that number."
   :group 'ggtags)
 
 (defcustom ggtags-global-output-format 'grep
-  "The output format for the 'global' command."
+  "Global output format: path, ctags, ctags-x, grep or cscope."
   :type '(choice (const path)
                  (const ctags)
                  (const ctags-x)
@@ -161,13 +180,14 @@ If an integer abbreviate only names longer than that number."
   :group 'ggtags)
 
 (defcustom ggtags-global-ignore-case nil
-  "Non-nil if Global should ignore case."
+  "Non-nil if Global should ignore case in the search pattern."
   :safe 'booleanp
   :type 'boolean
   :group 'ggtags)
 
 (defcustom ggtags-global-treat-text nil
-  "Non-nil if Global should include matches from text files."
+  "Non-nil if Global should include matches from text files.
+This affects `ggtags-find-file' and `ggtags-grep'."
   :safe 'booleanp
   :type 'boolean
   :group 'ggtags)
@@ -177,12 +197,32 @@ If an integer abbreviate only names longer than that number."
   :type 'number
   :group 'ggtags)
 
-(defcustom ggtags-global-next-error-hook nil
+(defcustom ggtags-global-history-length history-length
+  "Maximum number of items to keep in `ggtags-global-search-history'."
+  :type 'integer
+  :group 'ggtags)
+
+(defcustom ggtags-suppress-navigation-keys nil
+  "If non-nil key bindings in `ggtags-navigation-map' are suppressed."
+  :type 'boolean
+  :group 'ggtags)
+
+(defcustom ggtags-find-tag-hook nil
   "Hook run immediately after finding a tag."
-  :options '(reposition-window recenter)
+  :options '(recenter reposition-window)
   :type 'hook
   :group 'ggtags)
 
+(defcustom ggtags-show-definition-function #'ggtags-show-definition-default
+  "Function called by `ggtags-show-definition' to show definition.
+It is passed a list of definition candidates of the form:
+
+ (TEXT NAME FILE LINE)
+
+where TEXT is usually the source line of the definition."
+  :type 'function
+  :group 'ggtags)
+
 (defcustom ggtags-mode-sticky t
   "If non-nil enable Ggtags Mode in files visited."
   :safe 'booleanp
@@ -195,8 +235,8 @@ Users should change the value using `customize-variable' to
 properly update `ggtags-mode-map'."
   :set (lambda (sym value)
          (when (bound-and-true-p ggtags-mode-map)
-           (when-let (old (and (boundp sym) (symbol-value sym)))
-             (define-key ggtags-mode-map old nil))
+           (let ((old (and (boundp sym) (symbol-value sym))))
+             (and old (define-key ggtags-mode-map old nil)))
            (and value
                 (bound-and-true-p ggtags-mode-prefix-map)
                 (define-key ggtags-mode-map value ggtags-mode-prefix-map)))
@@ -220,11 +260,11 @@ properly update `ggtags-mode-map'."
 
 (defcustom ggtags-bounds-of-tag-function (lambda ()
                                            (bounds-of-thing-at-point 'symbol))
-  "Function to get the start and end locations of the tag at point."
+  "Function to get the start and end positions of the tag at point."
   :type 'function
   :group 'ggtags)
 
-(defvar ggtags-bug-url "https://github.com/leoliu/ggtags/issues")
+(defconst ggtags-bug-url "https://github.com/leoliu/ggtags/issues")
 
 (defvar ggtags-global-last-buffer nil)
 
@@ -249,14 +289,25 @@ properly update `ggtags-mode-map'."
 
 (defun ggtags-list-of-string-p (xs)
   "Return non-nil if XS is a list of strings."
-  (if (null xs)
-      t
-    (and (stringp (car xs))
-         (ggtags-list-of-string-p (cdr xs)))))
+  (cl-every #'stringp xs))
+
+(defun ggtags-forward-to-line (line)
+  "Move to line number LINE in current buffer."
+  (cl-check-type line (integer 1))
+  (save-restriction
+    (widen)
+    (goto-char (point-min))
+    (forward-line (1- line))))
+
+(defun ggtags-program-path (name)
+  (if ggtags-executable-directory
+      (expand-file-name name ggtags-executable-directory)
+    name))
 
 (defun ggtags-process-string (program &rest args)
   (with-temp-buffer
-    (let ((exit (apply #'process-file program nil t nil args))
+    (let ((exit (apply #'process-file
+                       (ggtags-program-path program) nil t nil args))
           (output (progn
                     (goto-char (point-max))
                     (skip-chars-backward " \t\n")
@@ -266,24 +317,26 @@ properly update `ggtags-mode-map'."
       output)))
 
 (defun ggtags-tag-at-point ()
-  (when-let (bounds (funcall ggtags-bounds-of-tag-function))
-    (buffer-substring (car bounds) (cdr bounds))))
+  (pcase (funcall ggtags-bounds-of-tag-function)
+    (`(,beg . ,end) (buffer-substring beg end))))
 
 ;;; Store for project info and settings
 
 (defvar ggtags-projects (make-hash-table :size 7 :test #'equal))
 
-(defstruct (ggtags-project (:constructor ggtags-project--make)
-                           (:copier nil)
-                           (:type vector)
-                           :named)
-  root tag-size has-refs has-path-style has-color dirty-p mtime timestamp)
+(cl-defstruct (ggtags-project (:constructor ggtags-project--make)
+                              (:copier nil)
+                              (:type vector)
+                              :named)
+  root config tag-size has-refs has-path-style has-color dirty-p mtime timestamp)
 
 (defun ggtags-make-project (root)
-  (check-type root string)
+  (cl-check-type root string)
   (pcase (nthcdr 5 (file-attributes (expand-file-name "GTAGS" root)))
     (`(,mtime ,_ ,tag-size . ,_)
      (let* ((default-directory (file-name-as-directory root))
+            (config (cl-some (lambda (c) (and (file-exists-p c) c))
+                             '(".globalrc" "gtags.conf")))
             (rtags-size (nth 7 (file-attributes "GRTAGS")))
             (has-refs
              (when rtags-size
@@ -294,16 +347,20 @@ properly update `ggtags-mode-map'."
             ;; http://thread.gmane.org/gmane.comp.gnu.global.bugs/1518
             (has-path-style
              (with-demoted-errors       ; in case `global' not found
-               (and (zerop (process-file "global" nil nil nil
+               (and (zerop (process-file (ggtags-program-path "global")
+                                         nil nil nil
                                          "--path-style" "shorter" "--help"))
                     'has-path-style)))
             ;; http://thread.gmane.org/gmane.comp.gnu.global.bugs/1542
             (has-color
              (with-demoted-errors
-               (and (zerop (process-file "global" nil nil nil "--color" "--help"))
+               (and (zerop (process-file (ggtags-program-path "global")
+                                         nil nil nil
+                                         "--color" "--help"))
                     'has-color))))
        (puthash default-directory
                 (ggtags-project--make :root default-directory
+                                      :config config
                                       :tag-size tag-size
                                       :has-refs has-refs
                                       :has-path-style has-path-style
@@ -334,8 +391,8 @@ Value is new modtime if updated."
   (pcase ggtags-oversize-limit
     (`nil nil)
     (`t t)
-    (size (when-let (project (or project (ggtags-find-project)))
-            (> (ggtags-project-tag-size project) size)))))
+    (size (let ((project (or project (ggtags-find-project))))
+            (and project (> (ggtags-project-tag-size project) size))))))
 
 (defvar-local ggtags-project-root 'unset
   "Internal variable for project root directory.")
@@ -350,21 +407,23 @@ Value is new modtime if updated."
               (ggtags-find-project))
           project)
       (setq ggtags-project-root
-            (or (ignore-errors (file-name-as-directory
-                                (concat (file-remote-p default-directory)
-                                        ;; Resolves symbolic links
-                                        (ggtags-process-string "global" "-pr"))))
-                ;; 'global -pr' resolves symlinks before checking
-                ;; the GTAGS file which could cause issues such as
+            (or (ignore-errors-unless-debug
+                  (file-name-as-directory
+                   (concat (file-remote-p default-directory)
+                           ;; Resolves symbolic links
+                           (ggtags-process-string "global" "-pr"))))
+                ;; 'global -pr' resolves symlinks before checking the
+                ;; GTAGS file which could cause issues such as
                 ;; https://github.com/leoliu/ggtags/issues/22, so
                 ;; let's help it out.
-                (when-let (gtags (locate-dominating-file
-                                  default-directory
-                                  (lambda (dir)
-                                    (file-regular-p (expand-file-name "GTAGS" dir)))))
+                ;;
+                ;; Note: `locate-dominating-file' doesn't accept
+                ;; function for NAME before 24.3.
+                (let ((dir (locate-dominating-file default-directory "GTAGS")))
                   ;; `file-truename' may strip the trailing '/' on
                   ;; remote hosts, see http://debbugs.gnu.org/16851
-                  (file-name-as-directory (file-truename gtags)))))
+                  (and dir (file-regular-p (expand-file-name "GTAGS" dir))
+                       (file-name-as-directory (file-truename dir))))))
       (when ggtags-project-root
         (if (gethash ggtags-project-root ggtags-projects)
             (ggtags-find-project)
@@ -403,13 +462,13 @@ Value is new modtime if updated."
                   (and (not (ggtags-project-has-refs (ggtags-find-project)))
                        (list "GTAGSLABEL=ctags"))))
          (envlist (delete-dups
-                   (loop for x in process-environment
-                         when (string-match
-                               "^\\(GTAGS[^=\n]*\\|MAKEOBJDIRPREFIX\\)=" x)
-                         ;; May have duplicates thus `delete-dups'.
-                         collect (concat (match-string 1 x)
-                                         "="
-                                         (getenv (match-string 1 x))))))
+                   (cl-loop for x in process-environment
+                            when (string-match
+                                  "^\\(GTAGS[^=\n]*\\|MAKEOBJDIRPREFIX\\)=" x)
+                            ;; May have duplicates thus `delete-dups'.
+                            collect (concat (match-string 1 x)
+                                            "="
+                                            (getenv (match-string 1 x))))))
          (help-form (format "y: save\nn: don't save\n=: diff\n?: help\n")))
     (add-dir-local-variable nil 'ggtags-process-environment envlist)
     ;; Remove trailing newlines by `add-dir-local-variable'.
@@ -436,14 +495,14 @@ Value is new modtime if updated."
     (save-buffer)
     (kill-buffer)
     (when buffer-file-name
-      (setq buffer-read-only val))
+      (read-only-mode (if val +1 -1)))
     (when (called-interactively-p 'interactive)
       (message "Project read-only-mode is %s" (if val "on" "off")))
     val))
 
 (defun ggtags-visit-project-root ()
   (interactive)
-  (ggtags-check-project)
+  (ggtags-ensure-project)
   (dired (ggtags-current-project-root)))
 
 (defmacro ggtags-with-current-project (&rest body)
@@ -462,17 +521,26 @@ Value is new modtime if updated."
                      (and ,gtagsroot (list (concat "GTAGSROOT=" ,gtagsroot)))
                      (and (ggtags-find-project)
                           (not (ggtags-project-has-refs (ggtags-find-project)))
-                          (list "GTAGSLABEL=ctags")))))
+                          (list "GTAGSLABEL=ctags"))
+                     (and ggtags-use-project-gtagsconf ,gtagsroot
+                          (ggtags-project-config (ggtags-find-project))
+                          (list (concat "GTAGSCONF="
+                                        (expand-file-name (ggtags-project-config
+                                                           (ggtags-find-project))
+                                                          ,gtagsroot)))))))
        (unwind-protect (save-current-buffer ,@body)
          (setq ggtags-project-root ,root)))))
 
 (defun ggtags-get-libpath ()
-  (when-let (path (ggtags-with-current-project (getenv "GTAGSLIBPATH")))
-    (mapcar (apply-partially #'concat (file-remote-p default-directory))
-            (split-string path (regexp-quote path-separator) t))))
+  (let ((path (ggtags-with-current-project (getenv "GTAGSLIBPATH"))))
+    (and path (mapcar (apply-partially #'concat (file-remote-p default-directory))
+                      (split-string path (regexp-quote path-separator) t)))))
 
 (defun ggtags-create-tags (root)
   "Create tag files (e.g. GTAGS) in directory ROOT.
+If file .globalrc or gtags.conf exists in ROOT, it will be used
+as configuration file per `ggtags-use-project-gtagsconf'.
+
 If file gtags.files exists in ROOT, it should be a list of source
 files to index, which can be used to speed gtags up in large
 source trees. See Info node `(global)gtags' for details."
@@ -482,9 +550,14 @@ source trees. See Info node `(global)gtags' for details."
     (setenv "GTAGSROOT" (expand-file-name
                          (directory-file-name (file-name-as-directory root))))
     (ggtags-with-current-project
-     (and (not (getenv "GTAGSLABEL"))
-          (yes-or-no-p "Use `ctags' backend? ")
-          (setenv "GTAGSLABEL" "ctags"))
+     (let ((conf (and ggtags-use-project-gtagsconf
+                      (cl-loop for name in '(".globalrc" "gtags.conf")
+                               for full = (expand-file-name name root)
+                               thereis (and (file-exists-p full) full)))))
+       (cond (conf (setenv "GTAGSCONF" conf))
+             ((and (not (getenv "GTAGSLABEL"))
+                   (yes-or-no-p "Use `ctags' backend? "))
+              (setenv "GTAGSLABEL" "ctags"))))
      (with-temp-message "`gtags' in progress..."
        (let ((default-directory (file-name-as-directory root)))
          (condition-case err
@@ -501,8 +574,7 @@ source trees. See Info node `(global)gtags' for details."
 
 (defun ggtags-update-tags (&optional force)
   "Update GNU Global tag database.
-Do nothing if GTAGS exceeds the oversize limit unless FORCE is
-non-nil."
+Do nothing if GTAGS exceeds the oversize limit unless FORCE."
   (interactive (progn
                  (ggtags-check-project)
                  ;; Mark project info expired.
@@ -519,46 +591,63 @@ non-nil."
 
 (defvar-local ggtags-completion-cache nil)
 
+;; See global/libutil/char.c
+;; (defconst ggtags-regexp-metachars "[][$()*+.?\\{}|^]")
+(defvar ggtags-completion-flag "")      ;internal use
+
 (defvar ggtags-completion-table
   (completion-table-dynamic
    (lambda (prefix)
-     (unless (equal prefix (car ggtags-completion-cache))
-       (setq ggtags-completion-cache
-             (cons prefix
-                   (ggtags-with-current-project
-                    (split-string
-                     (apply #'ggtags-process-string
-                            "global"
-                            ;; Note -c alone returns only definitions
-                            (if completion-ignore-case
-                                (list "--ignore-case" "-Tc" prefix)
-                              (list "-Tc" prefix)))
-                     "\n" t)))))
+     (let ((cache-key (concat prefix "$" ggtags-completion-flag)))
+       (unless (equal cache-key (car ggtags-completion-cache))
+         (setq ggtags-completion-cache
+               (cons cache-key
+                     (ignore-errors-unless-debug
+                       ;; May throw global: only name char is allowed
+                       ;; with -c option.
+                       (ggtags-with-current-project
+                        (split-string
+                         (apply #'ggtags-process-string
+                                "global"
+                                (append (and completion-ignore-case '("--ignore-case"))
+                                        ;; Note -c alone returns only definitions
+                                        (list (concat "-c" ggtags-completion-flag) prefix)))
+                         "\n" t)))))))
      (cdr ggtags-completion-cache))))
 
 (defun ggtags-completion-at-point ()
   "A function for `completion-at-point-functions'."
-  (when-let (bounds (funcall ggtags-bounds-of-tag-function))
-    (and (< (car bounds) (cdr bounds))
-         (list (car bounds) (cdr bounds) ggtags-completion-table))))
+  (pcase (funcall ggtags-bounds-of-tag-function)
+    (`(,beg . ,end)
+     (and (< beg end) (list beg end ggtags-completion-table)))))
 
-(defun ggtags-read-tag ()
+(defun ggtags-read-tag (&optional type confirm prompt require-match default)
   (ggtags-ensure-project)
-  (let ((default (ggtags-tag-at-point))
-        (completing-read-function ggtags-completing-read-function))
+  (let ((default (or default (ggtags-tag-at-point)))
+        (completing-read-function ggtags-completing-read-function)
+        (prompt (or prompt (capitalize (symbol-name (or type 'tag)))))
+        (ggtags-completion-flag (pcase type
+                                  (`(or nil definition) "T")
+                                  (`symbol "s")
+                                  (`reference "r")
+                                  (`id "I")
+                                  (`path "P")
+                                  ((pred stringp) type)
+                                  (_ ggtags-completion-flag))))
     (setq ggtags-current-tag-name
-          (cond (current-prefix-arg
+          (cond (confirm
                  (ggtags-update-tags)
                  (completing-read
-                  (format (if default "Tag (default %s): " "Tag: ") default)
-                  ggtags-completion-table nil t nil nil default))
+                  (format (if default "%s (default %s): " "%s: ") prompt default)
+                  ggtags-completion-table nil require-match nil nil default))
                 ((not default)
                  (user-error "No tag at point"))
                 (t (substring-no-properties default))))))
 
 (defun ggtags-global-build-command (cmd &rest args)
   ;; CMD can be definition, reference, symbol, grep, idutils
-  (let ((xs (append (list "global" "-v"
+  (let ((xs (append (list (shell-quote-argument (ggtags-program-path "global"))
+                          "-v"
                           (format "--result=%s" ggtags-global-output-format)
                           (and ggtags-global-ignore-case "--ignore-case")
                           (and (ggtags-find-project)
@@ -593,8 +682,8 @@ non-nil."
     (ring-insert find-tag-marker-ring ggtags-global-start-marker)
     (setq ggtags-global-start-marker t)))
 
-(defun ggtags-global-start (command &optional root)
-  (let* ((default-directory (or root (ggtags-current-project-root)))
+(defun ggtags-global-start (command &optional directory)
+  (let* ((default-directory (or directory (ggtags-current-project-root)))
          (split-window-preferred-function ggtags-split-window-function)
          ;; See http://debbugs.gnu.org/13594
          (display-buffer-overriding-action
@@ -602,15 +691,17 @@ non-nil."
                    ;; Appeared in emacs 24.4.
                    (fboundp 'display-buffer-no-window))
               (list #'display-buffer-no-window)
-            display-buffer-overriding-action)))
+            display-buffer-overriding-action))
+         (env ggtags-process-environment))
     (setq ggtags-global-start-marker (point-marker))
     (ggtags-navigation-mode +1)
     (setq ggtags-global-exit-status 0
           ggtags-global-match-count 0)
     (ggtags-update-tags)
     (ggtags-with-current-project
-     (setq ggtags-global-last-buffer
-           (compilation-start command 'ggtags-global-mode)))))
+     (with-current-buffer (compilation-start command 'ggtags-global-mode)
+       (setq-local ggtags-process-environment env)
+       (setq ggtags-global-last-buffer (current-buffer))))))
 
 (defun ggtags-find-tag-continue ()
   (interactive)
@@ -625,72 +716,80 @@ non-nil."
   (ggtags-global-start (apply #'ggtags-global-build-command cmd args)))
 
 ;;;###autoload
-(defun ggtags-find-tag-dwim (name &optional definition)
-  "Find definitions or references of tag NAME by context.
+(defun ggtags-find-tag-dwim (name &optional what)
+  "Find NAME by context.
 If point is at a definition tag, find references, and vice versa.
-With a prefix arg (non-nil DEFINITION) always find definitions."
-  (interactive (list (ggtags-read-tag) current-prefix-arg))
+If point is at a line that matches `ggtags-include-pattern', find
+the include file instead.
+
+When called interactively with a prefix arg, always find
+definition tags."
+  (interactive
+   (let ((include (and (not current-prefix-arg)
+                       ggtags-include-pattern
+                       (save-excursion
+                         (beginning-of-line)
+                         (if (functionp ggtags-include-pattern)
+                             (funcall ggtags-include-pattern)
+                           (and (looking-at (car ggtags-include-pattern))
+                                (match-string (cdr ggtags-include-pattern))))))))
+     (if include (list include 'include)
+       (list (ggtags-read-tag 'definition current-prefix-arg)
+             (and current-prefix-arg 'definition)))))
   (ggtags-check-project)     ; for `ggtags-current-project-root' below
-  (if (or definition
-          (not buffer-file-name)
-          (and (ggtags-find-project)
-               (not (ggtags-project-has-refs (ggtags-find-project)))))
-      (ggtags-find-tag 'definition name)
-    (ggtags-find-tag
-     (format "--from-here=%d:%s"
-             (line-number-at-pos)
-             (shell-quote-argument
-              ;; Note `ggtags-global-start' binds default-directory to
-              ;; project root.
-              (file-relative-name
-               buffer-file-name
-               (if (string-prefix-p (ggtags-current-project-root)
-                                    buffer-file-name)
-                   (ggtags-current-project-root)
-                 (locate-dominating-file buffer-file-name "GTAGS")))))
-     name)))
+  (cond
+   ((eq what 'include)
+    (ggtags-find-file name))
+   ((or (eq what 'definition)
+        (not buffer-file-name)
+        (and (ggtags-find-project)
+             (not (ggtags-project-has-refs (ggtags-find-project)))))
+    (ggtags-find-tag 'definition name))
+   (t (ggtags-find-tag
+       (format "--from-here=%d:%s"
+               (line-number-at-pos)
+               (shell-quote-argument
+                ;; Note `ggtags-global-start' binds default-directory to
+                ;; project root.
+                (file-relative-name
+                 buffer-file-name
+                 (if (string-prefix-p (ggtags-current-project-root)
+                                      buffer-file-name)
+                     (ggtags-current-project-root)
+                   (locate-dominating-file buffer-file-name "GTAGS")))))
+       name))))
 
 (defun ggtags-find-reference (name)
-  (interactive (list (ggtags-read-tag)))
+  (interactive (list (ggtags-read-tag 'reference current-prefix-arg)))
   (ggtags-find-tag 'reference name))
 
 (defun ggtags-find-other-symbol (name)
   "Find tag NAME that is a reference without a definition."
-  (interactive (list (ggtags-read-tag)))
+  (interactive (list (ggtags-read-tag 'symbol current-prefix-arg)))
   (ggtags-find-tag 'symbol name))
 
-(defun ggtags-read-string (prompt)
-  "Like `read-string' but handle default automatically."
-  (ggtags-ensure-project)
-  (let ((prompt (if (string-match ": *\\'" prompt)
-                    (substring prompt 0 (match-beginning 0))
-                  prompt))
-        (default (ggtags-tag-at-point)))
-    (read-string (format (if default "%s (default `%s'): " "%s: ")
-                         prompt default)
-                 nil nil default)))
-
 (defun ggtags-quote-pattern (pattern)
   (prin1-to-string (substring-no-properties pattern)))
 
+(defun ggtags-idutils-query (pattern)
+  (interactive (list (ggtags-read-tag 'id t)))
+  (ggtags-find-tag 'idutils "--" (ggtags-quote-pattern pattern)))
+
 (defun ggtags-grep (pattern &optional invert-match)
-  "Use `global --grep' to search for lines matching PATTERN.
+  "Grep for lines matching PATTERN.
 Invert the match when called with a prefix arg \\[universal-argument]."
-  (interactive (list (ggtags-read-string (if current-prefix-arg
-                                             "Inverted grep pattern"
-                                           "Grep pattern"))
+  (interactive (list (ggtags-read-tag 'definition 'confirm
+                                      (if current-prefix-arg
+                                          "Inverted grep pattern" "Grep pattern"))
                      current-prefix-arg))
   (ggtags-find-tag 'grep (and invert-match "--invert-match")
                    "--" (ggtags-quote-pattern pattern)))
 
-(defun ggtags-idutils-query (pattern)
-  (interactive (list (ggtags-read-string "ID query pattern")))
-  (ggtags-find-tag 'idutils "--" (ggtags-quote-pattern pattern)))
-
 (defun ggtags-find-file (pattern &optional invert-match)
-  (interactive (list (ggtags-read-string (if current-prefix-arg
-                                             "Inverted path pattern"
-                                           "Path pattern"))
+  (interactive (list (ggtags-read-tag 'path 'confirm (if current-prefix-arg
+                                                         "Inverted path pattern"
+                                                       "Path pattern")
+                                      nil (thing-at-point 'filename))
                      current-prefix-arg))
   (let ((ggtags-global-output-format 'path))
     (ggtags-find-tag 'path (and invert-match "--invert-match")
@@ -698,11 +797,12 @@ Invert the match when called with a prefix arg \\[universal-argument]."
 
 ;; NOTE: Coloured output in grep requested: http://goo.gl/Y9IcX
 (defun ggtags-find-tag-regexp (regexp directory)
-  "List tags matching REGEXP in DIRECTORY (default to project root)."
+  "List tags matching REGEXP in DIRECTORY (default to project root).
+When called interactively with a prefix, ask for the directory."
   (interactive
    (progn
      (ggtags-check-project)
-     (list (ggtags-read-string "POSIX regexp")
+     (list (ggtags-read-tag "" t "POSIX regexp")
            (if current-prefix-arg
                (read-directory-name "Directory: " nil nil t)
              (ggtags-current-project-root)))))
@@ -711,6 +811,8 @@ Invert the match when called with a prefix arg \\[universal-argument]."
    (ggtags-global-build-command nil nil "-l" "--" (ggtags-quote-pattern regexp))
    (file-name-as-directory directory)))
 
+(defvar ggtags-navigation-mode)
+
 (defun ggtags-query-replace (from to &optional delimited)
   "Query replace FROM with TO on files in the Global buffer.
 If not in navigation mode, do a grep on FROM first.
@@ -721,7 +823,7 @@ Global and Emacs."
    ;; Note: in 24.4 query-replace-read-args returns a list of 4 elements.
    (let ((args (query-replace-read-args "Query replace (regexp)" t t)))
      (list (nth 0 args) (nth 1 args) (nth 2 args))))
-  (unless (bound-and-true-p ggtags-navigation-mode)
+  (unless ggtags-navigation-mode
     (let ((ggtags-auto-jump-to-first-match nil))
       (ggtags-grep from)))
   (let ((file-form
@@ -741,16 +843,150 @@ Global and Emacs."
             (nreverse files))))
     (tags-query-replace from to delimited file-form)))
 
+(defvar ggtags-global-search-history nil)
+
+(defun ggtags-global-search-id (cmd directory)
+  (sha1 (concat directory (make-string 1 0) cmd)))
+
+(defun ggtags-global-current-search ()
+  ;; CMD DIR ENV LINE TEXT
+  (ggtags-ensure-global-buffer
+    (list (car compilation-arguments)
+          default-directory
+          ggtags-process-environment
+          (line-number-at-pos)
+          (buffer-substring-no-properties
+           (line-beginning-position) (line-end-position)))))
+
+(defun ggtags-global-rerun-search-1 (data)
+  (pcase data
+    (`(,cmd ,dir ,env ,line ,_text)
+     (with-current-buffer (let ((ggtags-auto-jump-to-first-match nil)
+                                ;; Switch current project to DIR.
+                                (default-directory dir)
+                                (ggtags-project-root dir)
+                                (ggtags-process-environment env))
+                            (ggtags-global-start cmd dir))
+       (add-hook 'compilation-finish-functions
+                 (lambda (buf _msg)
+                   (with-current-buffer buf
+                     (ggtags-forward-to-line line)
+                     (compile-goto-error)))
+                 nil t)))))
+
+(defvar-local ggtags-global-search-ewoc nil)
+(defvar ggtags-global-rerun-search-last nil)
+
+(defvar ggtags-global-rerun-search-map
+  (cl-labels
+      ((save ()
+             (setq ggtags-global-rerun-search-last
+                   (ewoc-data (ewoc-locate ggtags-global-search-ewoc))))
+       (next (arg)
+             (interactive "p")
+             (ewoc-goto-next ggtags-global-search-ewoc arg)
+             (save))
+       (prev (arg)
+             (interactive "p")
+             (ewoc-goto-prev ggtags-global-search-ewoc arg)
+             (save))
+       (quit ()
+             (interactive)
+             (quit-windows-on (ewoc-buffer ggtags-global-search-ewoc) t))
+       (done ()
+             (interactive)
+             (let ((node (ewoc-locate ggtags-global-search-ewoc)))
+               (when node
+                 (save)
+                 (quit)
+                 (ggtags-global-rerun-search-1 (cdr (ewoc-data node)))))))
+    (let ((m (make-sparse-keymap)))
+      (set-keymap-parent m special-mode-map)
+      (define-key m "p"    #'prev)
+      (define-key m "\M-p" #'prev)
+      (define-key m "n"    #'next)
+      (define-key m "\M-n" #'next)
+      (define-key m "r"    #'ggtags-save-to-register)
+      (define-key m "q"    #'quit)
+      (define-key m "\r"   #'done)
+      m)))
+
+(defvar bookmark-make-record-function)
+
+(defun ggtags-global-rerun-search ()
+  "Pop up a buffer to choose a past search to re-run.
+
+\\{ggtags-global-rerun-search-map}"
+  (interactive)
+  (or ggtags-global-search-history (user-error "No search history"))
+  (let ((split-window-preferred-function ggtags-split-window-function)
+        (inhibit-read-only t))
+    (pop-to-buffer "*Ggtags Search History*")
+    (erase-buffer)
+    (special-mode)
+    (use-local-map ggtags-global-rerun-search-map)
+    (setq-local ggtags-navigation-mode nil)
+    (setq-local bookmark-make-record-function #'ggtags-make-bookmark-record)
+    (setq truncate-lines t)
+    (cl-labels ((prop (s) (propertize s 'face 'minibuffer-prompt))
+                (pp (data)
+                    (pcase data
+                      (`(,_id ,cmd ,dir ,_env ,line ,text)
+                       (insert (prop " cmd: ") cmd "\n"
+                               (prop " dir: ") dir "\n"
+                               (prop "line: ") (number-to-string line) "\n"
+                               (prop "text: ") text "\n"
+                               (propertize (make-string 32 ?-) 'face 'shadow))))))
+      (setq ggtags-global-search-ewoc
+            (ewoc-create #'pp "Global search history keys:  n:next  p:prev  r:register  RET:choose\n")))
+    (dolist (data ggtags-global-search-history)
+      (ewoc-enter-last ggtags-global-search-ewoc data))
+    (and ggtags-global-rerun-search-last
+         (re-search-forward (cadr ggtags-global-rerun-search-last) nil t)
+         (ewoc-goto-node ggtags-global-search-ewoc
+                         (ewoc-locate ggtags-global-search-ewoc)))
+    (set-buffer-modified-p nil)
+    (fit-window-to-buffer nil (floor (frame-height) 2))))
+
+(defun ggtags-save-to-register (r)
+  "Save current search session to register R.
+Use \\[jump-to-register] to restore the search session."
+  (interactive (list (register-read-with-preview "Save search to register: ")))
+  (cl-labels ((prn (data)
+                   (pcase data
+                     (`(,command ,root ,_env ,line ,_)
+                      (princ (format "a ggtags search session `%s' in directory `%s' at line %d."
+                                     command root line))))))
+    (set-register r (registerv-make
+                     (if ggtags-global-search-ewoc
+                         (cdr (ewoc-data (ewoc-locate ggtags-global-search-ewoc)))
+                       (ggtags-global-current-search))
+                     :jump-func #'ggtags-global-rerun-search-1
+                     :print-func #'prn))))
+
+(defun ggtags-make-bookmark-record ()
+  `(,(and ggtags-current-tag-name (format "*ggtags %s*" ggtags-current-tag-name))
+    (ggtags-search . ,(if ggtags-global-search-ewoc
+                          (cdr (ewoc-data (ewoc-locate ggtags-global-search-ewoc)))
+                        (ggtags-global-current-search)))
+    (handler . ggtags-bookmark-jump)))
+
+(declare-function bookmark-prop-get "bookmark")
+
+(defun ggtags-bookmark-jump (bmk)
+  (ggtags-global-rerun-search-1 (bookmark-prop-get bmk 'ggtags-search)))
+
 (defun ggtags-delete-tag-files ()
-  "Delete the tag files generated by gtags."
+  "Delete the GTAGS, GRTAGS, GPATH etc. files generated by gtags."
   (interactive (ignore (ggtags-check-project)))
   (when (ggtags-current-project-root)
     (let* ((re (concat "\\`" (regexp-opt '("GPATH" "GRTAGS" "GTAGS" "ID")) "\\'"))
-           (files (loop for file in (directory-files (ggtags-current-project-root) t re)
-                        ;; Don't trust `directory-files'.
-                        when (let ((case-fold-search nil))
-                               (string-match-p re (file-name-nondirectory file)))
-                        collect file))
+           (files (cl-remove-if-not
+                   (lambda (file)
+                     ;; Don't trust `directory-files'.
+                     (let ((case-fold-search nil))
+                       (string-match-p re (file-name-nondirectory file))))
+                   (directory-files (ggtags-current-project-root) t re)))
            (buffer "*GTags File List*"))
       (or files (user-error "No tag files found"))
       (with-output-to-temp-buffer buffer
@@ -773,7 +1009,7 @@ Global and Emacs."
                    (list (read-file-name "Browse file: " nil nil t)
                          (read-number "Line: " 1))
                  (list buffer-file-name (line-number-at-pos))))
-  (check-type line integer)
+  (cl-check-type line integer)
   (or (and file (file-exists-p file)) (error "File `%s' doesn't exist" file))
   (ggtags-check-project)
   (or (file-exists-p (expand-file-name "HTML" (ggtags-current-project-root)))
@@ -855,15 +1091,15 @@ Global and Emacs."
               (setq tabulated-list-format
                     `[("ID" ,(max (1+ (floor (log counter 10))) 2)
                        (lambda (x y) (< (car x) (car y))))
-                      ("Buffer" ,(max (loop for m in elements
-                                            for b = (marker-buffer m)
-                                            maximize
-                                            (length (and b (buffer-name b))))
+                      ("Buffer" ,(max (cl-loop for m in elements
+                                               for b = (marker-buffer m)
+                                               maximize
+                                               (length (and b (buffer-name b))))
                                       6)
                        t :right-align t)
-                      ("Position" ,(max (loop for m in elements
-                                              for p = (or (marker-position m) 1)
-                                              maximize (1+ (floor (log p 10))))
+                      ("Position" ,(max (cl-loop for m in elements
+                                                 for p = (or (marker-position m) 1)
+                                                 maximize (1+ (floor (log p 10))))
                                         8)
                        (lambda (x y)
                          (< (string-to-number (aref (cadr x) 2))
@@ -885,7 +1121,7 @@ Global and Emacs."
                                               (funcall get-line x))
                                     (vector (number-to-string counter)
                                             "(dead)" "?" "?")))
-                          (decf counter)))
+                          (cl-decf counter)))
                       elements))))
     (setq tabulated-list-sort-key '("ID" . t))
     (tabulated-list-print)
@@ -929,9 +1165,8 @@ Global and Emacs."
 
 (defun ggtags-global-column (start)
   ;; START is the beginning position of source text.
-  (when-let (mbeg (text-property-any start (line-end-position) 'global-color t))
-    (setq ggtags-current-tag-name nil)
-    (- mbeg start)))
+  (let ((mbeg (text-property-any start (line-end-position) 'global-color t)))
+    (and mbeg (- mbeg start))))
 
 ;;; NOTE: Must not match the 'Global started at Mon Jun 3 10:24:13'
 ;;; line or `compilation-auto-jump' will jump there and fail. See
@@ -939,7 +1174,7 @@ Global and Emacs."
 ;;; `compilation-error-regexp-alist-alist'.
 (defvar ggtags-global-error-regexp-alist-alist
   (append
-   `((path "^\\(?:[^/\n]*/\\)?[^ )\t\n]+$" 0)
+   `((path "^\\(?:[^\"'\n]*/\\)?[^ )\t\n]+$" 0)
      ;; ACTIVE_ESCAPE  src/dialog.cc   172
      (ctags "^\\([^ \t\n]+\\)[ \t]+\\(.*?\\)[ \t]+\\([0-9]+\\)$"
             2 3 nil nil 2 (1 font-lock-function-name-face))
@@ -971,7 +1206,7 @@ Global and Emacs."
                             (1- (- wend wbeg)))))))
     (goto-char start)
     (while (and (> amount 0) (> end (point)))
-      (decf amount (funcall advance-word)))))
+      (cl-decf amount (funcall advance-word)))))
 
 (defun ggtags-abbreviate-files (start end)
   (goto-char start)
@@ -994,7 +1229,7 @@ Global and Emacs."
   (let ((buffer (or buffer (current-buffer))))
     (unless (get-buffer-window buffer)
       (let* ((split-window-preferred-function ggtags-split-window-function)
-             (w (display-buffer (current-buffer) '(nil (allow-no-window . t)))))
+             (w (display-buffer buffer '(nil (allow-no-window . t)))))
         (and w (compilation-set-window-height w))))))
 
 (defun ggtags-global-filter ()
@@ -1011,19 +1246,19 @@ Global and Emacs."
          "^ *Using \\(?:config file '.*\\|default configuration.\\)\n"
          compilation-filter-start t)
     (replace-match ""))
-  (incf ggtags-global-output-lines
-        (count-lines compilation-filter-start (point)))
-  (when (> ggtags-global-output-lines 5)
+  (cl-incf ggtags-global-output-lines
+           (count-lines compilation-filter-start (point)))
+  (when (and (> ggtags-global-output-lines 5) ggtags-navigation-mode)
     (ggtags-global--display-buffer))
   (make-local-variable 'ggtags-global-large-output)
   (when (> ggtags-global-output-lines ggtags-global-large-output)
-    (incf ggtags-global-large-output 500)
+    (cl-incf ggtags-global-large-output 500)
     (let ((message-log-max nil))
       (message "Output %d lines (Type `C-c C-k' to cancel)"
                ggtags-global-output-lines))))
 
-(defun ggtags-handle-single-match (buf _how)
-  (if (not (zerop ggtags-global-exit-status))
+(defun ggtags-handle-single-match (buf how)
+  (if (string-prefix-p "exited abnormally" how)
       ;; If exit abnormally display the buffer for inspection.
       (ggtags-global--display-buffer)
     (when (and ggtags-auto-jump-to-first-match
@@ -1057,7 +1292,7 @@ Global and Emacs."
               (list ggtags-global-output-format))
   (setq-local compilation-auto-jump-to-first-error
               ggtags-auto-jump-to-first-match)
-  (setq-local compilation-scroll-output 'first-error)
+  (setq-local compilation-scroll-output nil)
   ;; See `compilation-move-to-column' for details.
   (setq-local compilation-first-column 0)
   (setq-local compilation-error-screen-columns nil)
@@ -1072,6 +1307,7 @@ Global and Emacs."
   (jit-lock-register #'ggtags-abbreviate-files)
   (add-hook 'compilation-filter-hook 'ggtags-global-filter nil 'local)
   (add-hook 'compilation-finish-functions 'ggtags-handle-single-match nil t)
+  (setq-local bookmark-make-record-function #'ggtags-make-bookmark-record)
   (add-hook 'kill-buffer-hook (lambda () (ggtags-navigation-mode -1)) nil t))
 
 ;; NOTE: Need this to avoid putting menu items in
@@ -1085,6 +1321,9 @@ Global and Emacs."
     (define-key map "\M-{" 'ggtags-navigation-previous-file)
     (define-key map "\M->" 'ggtags-navigation-last-error)
     (define-key map "\M-<" 'ggtags-navigation-first-error)
+    ;; Note: shadows `isearch-forward-regexp' but it can be invoked
+    ;; with C-u C-s instead.
+    (define-key map "\C-\M-s" 'ggtags-navigation-isearch-forward)
     (define-key map "\C-c\C-k"
       (lambda () (interactive)
         (ggtags-ensure-global-buffer (kill-compilation))))
@@ -1097,10 +1336,6 @@ Global and Emacs."
 (defvar ggtags-mode-map-alist
   `((ggtags-navigation-mode . ,ggtags-navigation-map)))
 
-;; Higher priority for `ggtags-navigation-mode' to avoid being
-;; hijacked by modes such as `view-mode'.
-(add-to-list 'emulation-mode-map-alists 'ggtags-mode-map-alist)
-
 (defvar ggtags-navigation-mode-map
   (let ((map (make-sparse-keymap))
         (menu (make-sparse-keymap "GG-Navigation")))
@@ -1124,6 +1359,8 @@ Global and Emacs."
       '(menu-item "Previous file" ggtags-navigation-previous-file))
     (define-key menu [next-file]
       '(menu-item "Next file" ggtags-navigation-next-file))
+    (define-key menu [isearch-forward]
+      '(menu-item "Find match with isearch" ggtags-navigation-isearch-forward))
     (define-key menu [previous]
       '(menu-item "Previous match" previous-error))
     (define-key menu [next]
@@ -1132,15 +1369,22 @@ Global and Emacs."
 
 (defun ggtags-move-to-tag (&optional name)
   "Move to NAME tag in current line."
-  (let ((orig (point))
-        (tag (or name ggtags-current-tag-name)))
-    (beginning-of-line)
-    (if (and tag (re-search-forward
-                  (concat "\\_<" (regexp-quote tag) "\\_>")
-                  (line-end-position)
-                  t))
-        (goto-char (match-beginning 0))
-      (goto-char orig))))
+  (let ((tag (or name ggtags-current-tag-name)))
+    ;; Do nothing if on the tag already i.e. by `ggtags-global-column'.
+    (unless (or (not tag) (looking-at (concat (regexp-quote tag) "\\_>")))
+      (let ((orig (point))
+            (regexps (mapcar (lambda (fmtstr)
+                               (format fmtstr (regexp-quote tag)))
+                             '("\\_<%s\\_>" "%s\\_>" "%s"))))
+        (beginning-of-line)
+        (if (cl-loop for re in regexps
+                     ;; Note: tag might not agree with current
+                     ;; major-mode's symbol, so try harder. For
+                     ;; example, in `php-mode' $cacheBackend is a
+                     ;; symbol, but cacheBackend is a tag.
+                     thereis (re-search-forward re (line-end-position) t))
+            (goto-char (match-beginning 0))
+          (goto-char orig))))))
 
 (defun ggtags-navigation-mode-cleanup (&optional buf time)
   (let ((buf (or buf ggtags-global-last-buffer)))
@@ -1150,7 +1394,7 @@ Global and Emacs."
              (kill-compilation))
            (when (and (derived-mode-p 'ggtags-global-mode)
                       (get-buffer-window))
-             (quit-window nil (get-buffer-window)))
+             (quit-windows-on (current-buffer)))
            (and time (run-with-idle-timer time nil #'kill-buffer buf))))))
 
 (defun ggtags-navigation-mode-done ()
@@ -1195,6 +1439,17 @@ Global and Emacs."
     (compilation-previous-error 1)
     (compile-goto-error)))
 
+(defun ggtags-navigation-isearch-forward (&optional regexp-p)
+  (interactive "P")
+  (ggtags-ensure-global-buffer
+    (let ((saved (if visible-mode 1 -1)))
+      (visible-mode 1)
+      (with-selected-window (get-buffer-window (current-buffer))
+        (isearch-forward regexp-p)
+        (beginning-of-line)
+        (visible-mode saved)
+        (compile-goto-error)))))
+
 (defun ggtags-navigation-visible-mode (&optional arg)
   (interactive (list (or current-prefix-arg 'toggle)))
   (ggtags-ensure-global-buffer
@@ -1203,34 +1458,43 @@ Global and Emacs."
 (defvar ggtags-global-line-overlay nil)
 
 (defun ggtags-global-next-error-function ()
-  (ggtags-move-to-tag)
-  (ggtags-global-save-start-marker)
-  (and (ggtags-project-update-mtime-maybe)
-       (message "File `%s' is newer than GTAGS"
-                (file-name-nondirectory buffer-file-name)))
-  (and ggtags-mode-sticky (ggtags-mode 1))
-  (ignore-errors
-    (ggtags-ensure-global-buffer
-     (unless (overlayp ggtags-global-line-overlay)
-       (setq ggtags-global-line-overlay (make-overlay (point) (point)))
-       (overlay-put ggtags-global-line-overlay 'face 'ggtags-global-line))
-     (move-overlay ggtags-global-line-overlay
-                   (line-beginning-position) (line-end-position)
-                   (current-buffer))))
-  (run-hooks 'ggtags-global-next-error-hook))
+  (when (eq next-error-last-buffer ggtags-global-last-buffer)
+    (ggtags-move-to-tag)
+    (ggtags-global-save-start-marker)
+    (and (ggtags-project-update-mtime-maybe)
+         (message "File `%s' is newer than GTAGS"
+                  (file-name-nondirectory buffer-file-name)))
+    (and ggtags-mode-sticky (ggtags-mode 1))
+    (ignore-errors
+      (ggtags-ensure-global-buffer
+        (unless (overlayp ggtags-global-line-overlay)
+          (setq ggtags-global-line-overlay (make-overlay (point) (point)))
+          (overlay-put ggtags-global-line-overlay 'face 'ggtags-global-line))
+        (move-overlay ggtags-global-line-overlay
+                      (line-beginning-position) (line-end-position)
+                      (current-buffer))
+        ;; Update search history
+        (let ((id (ggtags-global-search-id (car compilation-arguments)
+                                           default-directory)))
+          (setq ggtags-global-search-history
+                (cl-remove id ggtags-global-search-history :test #'equal :key #'car))
+          (add-to-history 'ggtags-global-search-history
+                          (cons id (ggtags-global-current-search))
+                          ggtags-global-history-length))))
+    (run-hooks 'ggtags-find-tag-hook)))
 
 (define-minor-mode ggtags-navigation-mode nil
   :lighter
   (" GG[" (:eval
            (ignore-errors
              (ggtags-ensure-global-buffer
-              (let ((index (when (get-text-property (line-beginning-position)
-                                                    'compilation-message)
-                             ;; Assume the first match appears at line 5
-                             (- (line-number-at-pos) 4))))
-                `((:propertize ,(if index
-                                    (number-to-string (max index 0))
-                                  "?") face success) "/")))))
+               (let ((index (when (get-text-property (line-beginning-position)
+                                                     'compilation-message)
+                              ;; Assume the first match appears at line 5
+                              (- (line-number-at-pos) 4))))
+                 `((:propertize ,(if index
+                                     (number-to-string (max index 0))
+                                   "?") face success) "/")))))
    (:propertize (:eval (number-to-string ggtags-global-match-count))
                 face success)
    (:eval
@@ -1241,8 +1505,14 @@ Global and Emacs."
   :global t
   (if ggtags-navigation-mode
       (progn
+        ;; Higher priority for `ggtags-navigation-mode' to avoid being
+        ;; hijacked by modes such as `view-mode'.
+        (unless ggtags-suppress-navigation-keys
+          (add-to-list 'emulation-mode-map-alists 'ggtags-mode-map-alist))
         (add-hook 'next-error-hook 'ggtags-global-next-error-function)
         (add-hook 'minibuffer-setup-hook 'ggtags-minibuffer-setup-function))
+    (setq emulation-mode-map-alists
+          (delq 'ggtags-mode-map-alist emulation-mode-map-alists))
     (remove-hook 'next-error-hook 'ggtags-global-next-error-function)
     (remove-hook 'minibuffer-setup-hook 'ggtags-minibuffer-setup-function)))
 
@@ -1255,20 +1525,17 @@ Global and Emacs."
   (interactive "p")
   (ggtags-check-project)
   (let ((directories (cons (ggtags-current-project-root) (ggtags-get-libpath)))
-        (count 0)
-        (some (lambda (pred list)
-                (loop for x in list when (funcall pred x) return it))))
+        (count 0))
     (dolist (buf (buffer-list))
       (let ((file (and (buffer-live-p buf)
                        (not (eq buf (current-buffer)))
                        (buffer-file-name buf))))
-        (when (and file (funcall some
-                                 (lambda (dir)
+        (when (and file (cl-some (lambda (dir)
                                    ;; Don't use `file-in-directory-p'
                                    ;; to allow symbolic links.
                                    (string-prefix-p dir file))
                                  directories))
-          (and (kill-buffer buf) (incf count)))))
+          (and (kill-buffer buf) (cl-incf count)))))
     (and interactive
          (message "%d %s killed" count (if (= count 1) "buffer" "buffers")))))
 
@@ -1276,14 +1543,75 @@ Global and Emacs."
   (when (ggtags-find-project)
     (ggtags-project-update-mtime-maybe)
     ;; When oversize update on a per-save basis.
-    (when (and buffer-file-name (ggtags-project-oversize-p))
+    (when (and buffer-file-name
+               (or ggtags-global-always-update (ggtags-project-oversize-p)))
       (ggtags-with-current-project
-       (process-file "global" nil 0 nil "--single-update"
+       (process-file (ggtags-program-path "global") nil 0 nil "--single-update"
                      (file-relative-name buffer-file-name))))))
 
+(defun ggtags-global-output (buffer cmds callback &optional cutoff)
+  "Asynchronously pipe the output of running CMDS to BUFFER.
+When finished invoke CALLBACK in BUFFER with process exit status."
+  (or buffer (error "Output buffer required"))
+  (let* ((program (car cmds))
+         (args (cdr cmds))
+         (cutoff (and cutoff (+ cutoff (if (get-buffer buffer)
+                                           (with-current-buffer buffer
+                                             (line-number-at-pos (point-max)))
+                                         0))))
+         (proc (apply #'start-file-process program buffer program args))
+         (filter (lambda (proc string)
+                   (and (buffer-live-p (process-buffer proc))
+                        (with-current-buffer (process-buffer proc)
+                          (goto-char (process-mark proc))
+                          (insert string)
+                          (when (and (> (line-number-at-pos (point-max)) cutoff)
+                                     (process-live-p proc))
+                            (interrupt-process (current-buffer)))))))
+         (sentinel (lambda (proc _msg)
+                     (when (memq (process-status proc) '(exit signal))
+                       (with-current-buffer (process-buffer proc)
+                         (set-process-buffer proc nil)
+                         (funcall callback (process-exit-status proc)))))))
+    (set-process-query-on-exit-flag proc nil)
+    (and cutoff (set-process-filter proc filter))
+    (set-process-sentinel proc sentinel)
+    proc))
+
+(defun ggtags-show-definition-default (defs)
+  (let (message-log-max)
+    (message "%s%s" (or (caar defs) "[definition not found]")
+             (if (cdr defs) " [guess]" ""))))
+
+(defun ggtags-show-definition (name)
+  (interactive (list (ggtags-read-tag 'definition current-prefix-arg)))
+  (ggtags-check-project)
+  (let* ((re (cadr (assq 'grep ggtags-global-error-regexp-alist-alist)))
+         (current (current-buffer))
+         (buffer (get-buffer-create " *ggtags-definition*"))
+         (fn ggtags-show-definition-function)
+         (show (lambda (_status)
+                 (goto-char (point-min))
+                 (let ((defs (cl-loop while (re-search-forward re nil t)
+                                      collect (list (buffer-substring (1+ (match-end 2))
+                                                                      (line-end-position))
+                                                    name
+                                                    (match-string 1)
+                                                    (string-to-number (match-string 2))))))
+                   (kill-buffer buffer)
+                   (with-current-buffer current
+                     (funcall fn defs))))))
+    (ggtags-with-current-project
+     (ggtags-global-output
+      buffer
+      (list (ggtags-program-path "global")
+            "--result=grep" "--path-style=absolute" name)
+      show 100))))
+
 (defvar ggtags-mode-prefix-map
   (let ((m (make-sparse-keymap)))
-    (define-key m "\M-'" 'previous-error)
+    ;; Globally bound to `M-g p'.
+    ;; (define-key m "\M-'" 'previous-error)
     (define-key m (kbd "M-DEL") 'ggtags-delete-tag-files)
     (define-key m "\M-p" 'ggtags-prev-mark)
     (define-key m "\M-n" 'ggtags-next-mark)
@@ -1295,7 +1623,10 @@ Global and Emacs."
     (define-key m "\M-k" 'ggtags-kill-file-buffers)
     (define-key m "\M-h" 'ggtags-view-tag-history)
     (define-key m "\M-j" 'ggtags-visit-project-root)
+    (define-key m "\M-/" 'ggtags-global-rerun-search)
+    (define-key m (kbd "M-SPC") 'ggtags-save-to-register)
     (define-key m (kbd "M-%") 'ggtags-query-replace)
+    (define-key m "\M-?" 'ggtags-show-definition)
     m))
 
 (defvar ggtags-mode-map
@@ -1344,6 +1675,10 @@ Global and Emacs."
     (define-key menu [prev-mark]
       '(menu-item "Previous mark" ggtags-prev-mark))
     (define-key menu [sep1] menu-bar-separator)
+    (define-key menu [rerun-search]
+      '(menu-item "Rerun past search" ggtags-global-rerun-search))
+    (define-key menu [save-to-register]
+      '(menu-item "Save search session" ggtags-save-to-register))
     (define-key menu [previous-error]
       '(menu-item "Previous match" previous-error))
     (define-key menu [next-error]
@@ -1360,6 +1695,8 @@ Global and Emacs."
       '(menu-item "Find other symbol" ggtags-find-other-symbol))
     (define-key menu [find-tag-regexp]
       '(menu-item "Find tag matching regexp" ggtags-find-tag-regexp))
+    (define-key menu [show-definition]
+      '(menu-item "Show definition" ggtags-show-definition))
     (define-key menu [find-reference]
       '(menu-item "Find reference" ggtags-find-reference))
     (define-key menu [find-tag-continue]
@@ -1374,6 +1711,29 @@ Global and Emacs."
                   :visible (not (ggtags-find-project))))
     map))
 
+(defvar ggtags-mode-line-project-keymap
+  (let ((map (make-sparse-keymap)))
+    (define-key map [mode-line mouse-1] 'ggtags-visit-project-root)
+    map))
+
+(put 'ggtags-mode-line-project-name 'risky-local-variable t)
+(defvar ggtags-mode-line-project-name
+  '("[" (:eval (let ((name (if (stringp ggtags-project-root)
+                               (file-name-nondirectory
+                                (directory-file-name ggtags-project-root))
+                             "?")))
+                 (propertize
+                  name 'face compilation-info-face
+                  'help-echo (if (stringp ggtags-project-root)
+                                 (concat "mouse-1 to visit " ggtags-project-root)
+                               "mouse-1 to set project")
+                  'mouse-face 'mode-line-highlight
+                  'keymap ggtags-mode-line-project-keymap)))
+    "]")
+  "Mode line construct for displaying current project name.
+The value is the name of the project root directory. Setting it
+to nil disables displaying this information.")
+
 ;;;###autoload
 (define-minor-mode ggtags-mode nil
   :lighter (:eval (if ggtags-navigation-mode "" " GG"))
@@ -1386,9 +1746,16 @@ Global and Emacs."
         (add-hook 'after-save-hook 'ggtags-after-save-function nil t)
         ;; Append to serve as a fallback method.
         (add-hook 'completion-at-point-functions
-                  #'ggtags-completion-at-point t t))
+                  #'ggtags-completion-at-point t t)
+        (unless (memq 'ggtags-mode-line-project-name
+                      mode-line-buffer-identification)
+          (setq mode-line-buffer-identification
+                (append mode-line-buffer-identification
+                        '(ggtags-mode-line-project-name)))))
     (remove-hook 'after-save-hook 'ggtags-after-save-function t)
     (remove-hook 'completion-at-point-functions #'ggtags-completion-at-point t)
+    (setq mode-line-buffer-identification
+          (delq 'ggtags-mode-line-project-name mode-line-buffer-identification))
     (and (overlayp ggtags-highlight-tag-overlay)
          (delete-overlay ggtags-highlight-tag-overlay))
     (setq ggtags-highlight-tag-overlay nil)))
@@ -1428,12 +1795,9 @@ Global and Emacs."
         ;; Overlay matches current tag so do nothing.
         nil)
        ((and bounds (let ((completion-ignore-case nil))
-                      (ignore-errors
-                        ;; May throw: global: only name char is
-                        ;; allowed with -c option
-                        (test-completion
-                         (buffer-substring (car bounds) (cdr bounds))
-                         ggtags-completion-table))))
+                      (test-completion
+                       (buffer-substring (car bounds) (cdr bounds))
+                       ggtags-completion-table)))
         (move-overlay o (car bounds) (cdr bounds) (current-buffer))
         (overlay-put o 'category 'ggtags-active-tag))
        (t (move-overlay o
@@ -1445,31 +1809,29 @@ Global and Emacs."
 ;;; imenu
 
 (defun ggtags-goto-imenu-index (name line &rest _args)
-  (save-restriction
-    (widen)
-    (goto-char (point-min))
-    (forward-line (1- line))
-    (ggtags-move-to-tag name)))
+  (ggtags-forward-to-line line)
+  (ggtags-move-to-tag name))
 
 ;;;###autoload
 (defun ggtags-build-imenu-index ()
   "A function suitable for `imenu-create-index-function'."
-  (when-let (file (and buffer-file-name (file-relative-name buffer-file-name)))
-    (with-temp-buffer
-      (when (with-demoted-errors
-              (zerop (ggtags-with-current-project
-                      (process-file "global" nil t nil "-x" "-f" file))))
-        (goto-char (point-min))
-        (loop while (re-search-forward
-                     "^\\([^ \t]+\\)[ \t]+\\([0-9]+\\)" nil t)
-              collect (list (match-string 1)
-                            (string-to-number (match-string 2))
-                            'ggtags-goto-imenu-index))))))
+  (let ((file (and buffer-file-name (file-relative-name buffer-file-name))))
+    (and file (with-temp-buffer
+                (when (with-demoted-errors
+                        (zerop (ggtags-with-current-project
+                                (process-file (ggtags-program-path "global")
+                                              nil t nil "-x" "-f" file))))
+                  (goto-char (point-min))
+                  (cl-loop while (re-search-forward
+                                  "^\\([^ \t]+\\)[ \t]+\\([0-9]+\\)" nil t)
+                           collect (list (match-string 1)
+                                         (string-to-number (match-string 2))
+                                         'ggtags-goto-imenu-index)))))))
 
 ;;; hippie-expand
 
 ;;;###autoload
-(defun try-complete-ggtags-tag (old)
+(defun ggtags-try-complete-tag (old)
   "A function suitable for `hippie-expand-try-functions-list'."
   (with-no-warnings                     ; to avoid loading hippie-exp
     (unless old
@@ -1496,10 +1858,5 @@ Global and Emacs."
   (unload-feature 'ggtags force)
   (require 'ggtags))
 
-(defun ggtags-unload-function ()
-  (setq emulation-mode-map-alists
-        (delq 'ggtags-mode-map-alist emulation-mode-map-alists))
-  nil)
-
 (provide 'ggtags)
 ;;; ggtags.el ends here