[debbugs-tracker] bug#12297: closed (show-paren-mode only recognizes show-paren-delay value on activation)

[GNU bug Tracking System]

Your message dated Wed, 29 Aug 2012 21:45:26 -0400
with message-id <address@hidden>
and subject line Re: bug#12297: show-paren-mode only recognizes 
show-paren-delay value on activation
has caused the debbugs.gnu.org bug report #12297,
regarding show-paren-mode only recognizes show-paren-delay value on activation
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
12297: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=12297
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> show-paren-mode only recognizes show-paren-delay value on activation
>
>
>
>
> Date: 
>
> Tue, 28 Aug 2012 18:47:38 -0400
>
>
>
>
> Activating show-paren-mode sets the delay time to show-paren-delay initially, but altering that value later has no effect. i.e., after:
>
> (show-paren-mode)
> (setq show-paren-delay 0)
>
> The timer is still set to the original value .125, until the mode is re-activated.
> This should maybe be noted in the variable description, or there should be a dedicated function to set the new value, e.g. by turning line 125 of paren.el into a separate function:
>
> (setq show-paren-idle-timer (run-with-idle-timer
>                    show-paren-delay t
>                    'show-paren-function))
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#12297: show-paren-mode only recognizes show-paren-delay value	on activation
>
>
>
>
> Date: 
>
> Wed, 29 Aug 2012 21:45:26 -0400
>
>
>
>
> User-agent: 
>
> Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
>
>
>
>
> Version: 24.3
>
> Roy Crihfield wrote:
>
> > Activating show-paren-mode sets the delay time to show-paren-delay
> > initially, but altering that value later has no effect.
>
> OK, fixed in current trunk.
>
> *** lisp/paren.el       2012-04-09 13:05:48 +0000
> --- lisp/paren.el       2012-08-30 01:37:52 +0000
> ***************
> *** 52,59 ****
>     :group 'paren-showing)
>   
>   (defcustom show-paren-delay 0.125
> !   "Time in seconds to delay before showing a matching paren."
>     :type '(number :tag "seconds")
>     :group 'paren-showing)
>   
>   (defcustom show-paren-priority 1000
> --- 52,68 ----
>     :group 'paren-showing)
>   
>   (defcustom show-paren-delay 0.125
> !   "Time in seconds to delay before showing a matching paren.
> ! If you change this without using customize while `show-paren-mode' is
> ! active, you must toggle the mode off and on again for this to take effect."
>     :type '(number :tag "seconds")
> +   :initialize 'custom-initialize-default
> +   :set (lambda (sym val)
> +        (if (not show-paren-mode)
> +            (set sym val)
> +          (show-paren-mode -1)
> +          (set sym val)
> +          (show-paren-mode 1)))
>     :group 'paren-showing)
>   
>   (defcustom show-paren-priority 1000
>
>
>
> --- End Message ---