[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Couple of useful (?) patches



At Tue, 05 Apr 2011 22:20:50 -0400,
Vitaly Mayatskikh wrote:
> 
> Hi!
> 
> This is follow up to recent issues claimed by Dave Abrahams on wl-en
> list. Indeed, I agree with him, here're fixes:
> 
> First issue is weird MIME buttons appear in quoted reply. I see no
> reason to keep text representation of buttons in reply, do you? To
> work that around I propose to turn on visibility of buttons using
> relatively recent mime-view-buttons-visible patch (was it a year ago?
> or two?) just before reply is going to be rendered. Solution is pretty
> dumb: the whole pre-rendered cache is ripping out now, but... It
> works, and who cares about cache? ;)
> 
> Index: wl-summary.el
> ===================================================================
> RCS file: /cvs/root/wanderlust/wl/wl-summary.el,v
> retrieving revision 1.450
> diff -p -u -w -r1.450 wl-summary.el
> --- wl-summary.el	18 Oct 2010 07:43:32 -0000	1.450
> +++ wl-summary.el	6 Apr 2011 02:06:43 -0000
> @@ -4031,6 +4031,9 @@ Return t if message exists."
>  
>  (defun wl-summary-reply-with-citation (&optional arg)
>    (interactive "P")
> +  (let (mime-view-buttons-visible)
> +    (wl-message-buffer-cache-clean-up)
> +    (wl-summary-redisplay))
>    (when (wl-summary-reply arg t)
>      (goto-char (point-max))
>      (wl-draft-yank-original)
>
> Please, test these patches. If you'll not find any problems, I'm about
> to commit them.
> 

Based on that I wrote

(defadvice wl-summary-reply-with-citation 
  (around wl-summary-reply-sans-mime-buttons activate )
    (let (mime-view-buttons-visible)
      (wl-message-buffer-cache-clean-up)
      (wl-summary-redisplay)
      ad-do-it))

But what I found was that even if I had selected a range of text in
the message buffer, the reply cited everything in the message.  So I
think this one needs some work unless my advice works differently from
your patch somehow.


-- 
Dave Abrahams
BoostPro Computing
http://www.boostpro.com