Gnus development mailing list
 help / color / mirror / Atom feed
From: Pete Kazmier <pete@kazmier.com>
Subject: Re: Treatment for gnus-article-wash-html?
Date: Thu, 02 Jan 2003 16:44:24 -0500	[thread overview]
Message-ID: <m3smwb1b2f.fsf@coco.kazmier.com> (raw)
In-Reply-To: <m3r8bvb838.fsf@quimbies.gnus.org>

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

Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
> Hm.  That's not supposed to happen...  I've done some minor tweaks
> to the HTML washing function, so could you try updating to the
> latest CVS and see whether the problem still exists?  If it does,
> I'll poke around some more...

The problem still exists, i.e. if I specify a string list as the value
of the treatment variable, it gets applied to all parts of the article
(including the headers).

After looking at the code (bear in mind that I'm a complete newbie), I
think I have corrected the problem. I was able to fix this behavior
with the following patch.

Thanks,
Pete


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Patch to fix treatment of headers --]
[-- Type: text/x-patch, Size: 664 bytes --]

Index: lisp/gnus-art.el
===================================================================
RCS file: /usr/local/cvsroot/gnus/lisp/gnus-art.el,v
retrieving revision 6.264
diff -u -r6.264 gnus-art.el
--- lisp/gnus-art.el	2003/01/02 20:45:14	6.264
+++ lisp/gnus-art.el	2003/01/02 21:30:06
@@ -6253,6 +6253,8 @@
   (cond
    ((null val)
     nil)
+   (condition
+    (eq condition val))
    ((and (listp val)
 	 (stringp (car val)))
     (apply 'gnus-or (mapcar `(lambda (s)
@@ -6271,8 +6273,6 @@
 	(equal (car val) type))
        (t
 	(error "%S is not a valid predicate" pred)))))
-   (condition
-    (eq condition val))
    ((eq val t)
     t)
    ((eq val 'head)

  reply	other threads:[~2003-01-02 21:44 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-02 19:26 pete-temp-12-29-2002
2003-01-02 19:35 ` Lars Magne Ingebrigtsen
2003-01-02 20:21   ` pete-temp-12-29-2002
2003-01-02 20:38     ` Lars Magne Ingebrigtsen
2003-01-02 21:44       ` Pete Kazmier [this message]
2003-01-02 21:48         ` Lars Magne Ingebrigtsen
2003-01-02 22:07 ` pete-temp-12-29-2002

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=m3smwb1b2f.fsf@coco.kazmier.com \
    --to=pete@kazmier.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).