Bug 52553 - [PATCH] Newsgroups header must not contain spaces (so let's remove them)
Summary: [PATCH] Newsgroups header must not contain spaces (so let's remove them)
Status: RESOLVED FIXED
Alias: None
Product: knode
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-01-03 14:26 UTC by Malte S. Stretz
Modified: 2003-01-06 02:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch against kncomposer.cpp (600 bytes, patch)
2003-01-03 14:28 UTC, Malte S. Stretz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Malte S. Stretz 2003-01-03 14:26:55 UTC
Version:            (using KDE KDE 3.0.99)
Installed from:    Compiled From Sources
Compiler:          gcc 3.2.1 
OS:          Linux

Hi,

it seems like the RFC does not allow spaces in the Newsgroups header. I tend to type a space after the comma if I insert the groups manually (eg. "de.test, linux.test" instead of "de.test,linux.test"). The news server will refuse those messages with something like '441 437 Whitespaces in "Newsgroups" header'. That's quite annoying, especially in combination with bug 44319.

To make mine (and probably others') life much easier, the attached patch removes all whitespaces from the Newsgroups header. Simple but efficient :o)

Cheers,
Malte
Comment 1 Malte S. Stretz 2003-01-03 14:28:02 UTC
Created attachment 690 [details]
Patch against kncomposer.cpp

P.S.: This is the first line I've ever written in C++ so I don't know if it's
the right place to remove the whitespaces and if it's 100% correct. Oh, the
patch is against the 3.1-rc5 sources.
Comment 2 Christian Gebauer 2003-01-06 02:09:18 UTC
your patch is correct and I applied it,  
thanks for the report