After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 756900 - evolution mangles quoted messages when switching between "preformatted" and "normal"
evolution mangles quoted messages when switching between "preformatted" and "...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Composer
3.16.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Tomas Popela
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2015-10-21 10:38 UTC by Jeff Layton
Modified: 2015-10-22 08:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
sample message for reproducing problem (4.13 KB, application/mbox)
2015-10-21 10:38 UTC, Jeff Layton
Details

Description Jeff Layton 2015-10-21 10:38:26 UTC
Created attachment 313797 [details]
sample message for reproducing problem

Bug 756883 is a request for a setting to have evolution _not_ line wrap quoted messages in replies as that mangles things like patches that are inlined in the mail.

One of the proffered workarounds was to select the quoted text and switch it to be "Preformatted", but that leaves the resulting quoted message with spurious '>' characters sprinkled all over it.

I'm running evolution-3.16.5-3.fc22.x86_64 from Fedora. To reproduce, import the attached message into your mailbox. Select the imported message and click the "Reply" button. You'll notice that some of the C comments in the quoted reply are now mangled due to line wrapping. Hit ctrl-a to select all, and then change the paragraph style from "Normal" to "Preformatted". Many of the functions will now have spurious '>' characters in them. For example:

> +static inline void
> +lease_notifier_chain_init(void)
> +{
> +> 	> srcu_init_notifier_head(&lease_notifier_chain);
> +}
> +
> +static inline void
> +setlease_notifier(long arg, struct file_lock *lease)
> +{
> +> 	> if (arg != F_UNLCK)
> +> 	> 	> srcu_notifier_call_chain(&lease_notifier_chain, arg, lease);
> +}
> +#else /* !IS_ENABLED(CONFIG_SRCU) */
> +static inline void
Comment 1 Milan Crha 2015-10-22 08:46:10 UTC
Thanks for a  bug report. I tried to reproduce this with the latest development version and there are not added any extra quotation marks in the text. This development version will be 3.19.1 soon.

I can reproduce this with evolution 3.18.1. I could reproduce it even with a gnome-3-18 branch checkout at commit 3230364, but when I updated to commit b8353ac, then the issue had been fixed.

I do not know which exact commit fixed it, but the version 3.18.2+ will be fine.