all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alex Branham <alex.branham@gmail.com>
To: Robert Pluim <rpluim@gmail.com>
Cc: 18426@debbugs.gnu.org
Subject: bug#18426: [PATCH] Increase default size of imenu-auto-rescan-maxout
Date: Fri, 31 Aug 2018 08:07:12 -0500	[thread overview]
Message-ID: <87in3qsn0f.fsf@gmail.com> (raw)
In-Reply-To: <87bm9j2cy5.fsf@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2712 bytes --]


On Fri 31 Aug 2018 at 02:49, Robert Pluim <rpluim@gmail.com> wrote:

> You could probably put both of them in emacs-26, although thatʼs up to
> Eli. In any case, emacs-26 gets merged to master regularly, so the doc
> change could definitely go there.

Great, I've attached a patch that bunches it all together for the emacs-26 branch.

Thanks,
Alex

------------------------------------------------------------

From 1fd71c5195eb3c32e1da161770fbf4dd820d8e31 Mon Sep 17 00:00:00 2001
From: Alex Branham <alex.branham@gmail.com>
Date: Fri, 31 Aug 2018 08:05:06 -0500
Subject: [PATCH] Increase default value for imenu-auto-rescan-maxout

* lisp/imenu.el (imenu-auto-rescan-maxout): Increase default value to
  600000. (Bug#18426)
* doc/emacs/programs.texi (imenu-auto-rescan-maxout): Add
  documentation for imenu-auto-rescan-maxout.
---
 doc/emacs/programs.texi | 4 ++++
 etc/NEWS                | 4 ++++
 lisp/imenu.el           | 8 ++++----
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/doc/emacs/programs.texi b/doc/emacs/programs.texi
index 138f82a6bf..46711aaf30 100644
--- a/doc/emacs/programs.texi
+++ b/doc/emacs/programs.texi
@@ -308,6 +308,10 @@ Imenu
 a non-@code{nil} value.  There is no need to rescan because of small
 changes in the text.
 
+@vindex imenu-auto-rescan-maxout
+  @code{imenu-auto-rescan} will be disabled in buffers that are larger
+than @code{imenu-auto-rescan-maxout} in bytes.
+
 @vindex imenu-sort-function
   You can customize the way the menus are sorted by setting the
 variable @code{imenu-sort-function}.  By default, names are ordered as
diff --git a/etc/NEWS b/etc/NEWS
index f575d4dd00..90f98041be 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -50,6 +50,10 @@ often cause crashes.  Set it to nil if you really need those fonts.
 ---
 *** New toggle 'ibuffer-do-toggle-lock', bound to 'L'.
 
+** Imenu
+---
+*** The value for 'imenu-auto-rescan-maxout' has been increased to 600000.
+
 ** Gnus
 
 ---
diff --git a/lisp/imenu.el b/lisp/imenu.el
index 89114524eb..6185debc83 100644
--- a/lisp/imenu.el
+++ b/lisp/imenu.el
@@ -96,11 +96,11 @@ imenu-auto-rescan
   :type 'boolean
   :group 'imenu)
 
-(defcustom imenu-auto-rescan-maxout 60000
-  "Imenu auto-rescan is disabled in buffers larger than this size (in bytes).
-This variable is buffer-local."
+(defcustom imenu-auto-rescan-maxout 600000
+  "Imenu auto-rescan is disabled in buffers larger than this size (in bytes)."
   :type 'integer
-  :group 'imenu)
+  :group 'imenu
+  :version "27.1")
 
 (defvar imenu-always-use-completion-buffer-p nil)
 (make-obsolete-variable 'imenu-always-use-completion-buffer-p
-- 
2.18.0


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Increase-default-value-for-imenu-auto-rescan-maxout.patch --]
[-- Type: text/x-patch, Size: 2216 bytes --]

From 1fd71c5195eb3c32e1da161770fbf4dd820d8e31 Mon Sep 17 00:00:00 2001
From: Alex Branham <alex.branham@gmail.com>
Date: Fri, 31 Aug 2018 08:05:06 -0500
Subject: [PATCH] Increase default value for imenu-auto-rescan-maxout

* lisp/imenu.el (imenu-auto-rescan-maxout): Increase default value to
  600000. (Bug#18426)
* doc/emacs/programs.texi (imenu-auto-rescan-maxout): Add
  documentation for imenu-auto-rescan-maxout.
---
 doc/emacs/programs.texi | 4 ++++
 etc/NEWS                | 4 ++++
 lisp/imenu.el           | 8 ++++----
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/doc/emacs/programs.texi b/doc/emacs/programs.texi
index 138f82a6bf..46711aaf30 100644
--- a/doc/emacs/programs.texi
+++ b/doc/emacs/programs.texi
@@ -308,6 +308,10 @@ Imenu
 a non-@code{nil} value.  There is no need to rescan because of small
 changes in the text.
 
+@vindex imenu-auto-rescan-maxout
+  @code{imenu-auto-rescan} will be disabled in buffers that are larger
+than @code{imenu-auto-rescan-maxout} in bytes.
+
 @vindex imenu-sort-function
   You can customize the way the menus are sorted by setting the
 variable @code{imenu-sort-function}.  By default, names are ordered as
diff --git a/etc/NEWS b/etc/NEWS
index f575d4dd00..90f98041be 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -50,6 +50,10 @@ often cause crashes.  Set it to nil if you really need those fonts.
 ---
 *** New toggle 'ibuffer-do-toggle-lock', bound to 'L'.
 
+** Imenu
+---
+*** The value for 'imenu-auto-rescan-maxout' has been increased to 600000.
+
 ** Gnus
 
 ---
diff --git a/lisp/imenu.el b/lisp/imenu.el
index 89114524eb..6185debc83 100644
--- a/lisp/imenu.el
+++ b/lisp/imenu.el
@@ -96,11 +96,11 @@ imenu-auto-rescan
   :type 'boolean
   :group 'imenu)
 
-(defcustom imenu-auto-rescan-maxout 60000
-  "Imenu auto-rescan is disabled in buffers larger than this size (in bytes).
-This variable is buffer-local."
+(defcustom imenu-auto-rescan-maxout 600000
+  "Imenu auto-rescan is disabled in buffers larger than this size (in bytes)."
   :type 'integer
-  :group 'imenu)
+  :group 'imenu
+  :version "27.1")
 
 (defvar imenu-always-use-completion-buffer-p nil)
 (make-obsolete-variable 'imenu-always-use-completion-buffer-p
-- 
2.18.0


  reply	other threads:[~2018-08-31 13:07 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-08 16:13 bug#18426: 24.3; imenu-auto-rescan-maxout not mentioned in manual Ethan Glasser-Camp
2014-09-09  0:38 ` Stefan Monnier
2018-08-30 21:46 ` bug#18426: [PATCH] Increase default size of imenu-auto-rescan-maxout Alex Branham
2018-08-30 21:55   ` Alex Branham
2018-08-31  7:49     ` Robert Pluim
2018-08-31 13:07       ` Alex Branham [this message]
2018-08-31 13:12         ` Robert Pluim
2018-08-31 13:15           ` Alex Branham
2018-09-06 15:09             ` Alex Branham
2018-09-06 15:16               ` Robert Pluim
2018-09-06 18:43                 ` Eli Zaretskii
2018-09-12 13:33                   ` Robert Pluim
2018-09-12 14:07                     ` Alex Branham
2018-09-12 14:29                     ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87in3qsn0f.fsf@gmail.com \
    --to=alex.branham@gmail.com \
    --cc=18426@debbugs.gnu.org \
    --cc=rpluim@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.