[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    [PATCH] fix for 61305 - After applying filters,
From:       Andreas Gungl <a.gungl () gmx ! de>
Date:       2004-02-28 21:27:25
Message-ID: 200402282227.31112 () gungl-dd ! de
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

please, can someone being familiar with the filtering stuff in KMail review 
the patch? I've tested it and it fixes the behavior described in the bug 
report http://bugs.kde.org/show_bug.cgi?id=61305 but I'm nor sure regarding 
possible side effects.

TIA,
Andreas
- -- 
    ~
  ' v '
 //   \\
/(     )\  Powered by Penguin.
  ^ ' ^

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2-rc1-SuSE (GNU/Linux)

iD8DBQFAQQe9VhjiFd4beU8RAvyzAKCopeqTW5YchaMB4EQRqjPNkCCf5QCgmrzU
+RHD+uSRKUcf4bJedO4eCcA=
=uqJP
-----END PGP SIGNATURE-----

["kmailfilteraction.diff" (text/x-diff)]

Index: kmfilteraction.cpp
===================================================================
RCS file: /home/kde/kdepim/kmail/kmfilteraction.cpp,v
retrieving revision 1.115
diff -u -3 -p -r1.115 kmfilteraction.cpp
--- kmfilteraction.cpp	5 Feb 2004 22:35:50 -0000	1.115
+++ kmfilteraction.cpp	28 Feb 2004 21:23:21 -0000
@@ -497,8 +497,12 @@ KMFilterAction::ReturnCode KMFilterActio
 		      withOutput ? KProcess::Stdout : KProcess::NoCommunication ) )
     return ErrorButGoOn;
 
-  if ( !shProc.normalExit() || shProc.exitStatus() != 0 )
+  if ( !shProc.normalExit() || shProc.exitStatus() != 0 ) {
+    // eat the output to avoid acummulation for following processes
+    if ( withOutput ) 
+      kmkernel->getCollectedStdOut( &shProc );
     return ErrorButGoOn;
+  }
 
   if ( withOutput ) {
     // read altered message:


_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread]