Thomas,

your fix contains the following change:

> +                if ($q == $q_level 
> +                    && strlen($line[$last]) > 1  // don't hit if line only 
> consist of one single white space 

What if a quoted blank line contains more than one single WSP?

Cheers,
-- 
Michael Heydekamp
Co-Admin freexp.de
Düsseldorf/Germany


Am 19.01.2013 01:17, schrieb Michael Heydekamp:
> Am 18.01.2013 21:31, schrieb Thomas Bruederli:
> 
>> OK, got it:
>>
>> Quoted empty lines "> " have a trailing space
> 
> Well, not always, but quite often, yes.
> 
>> which, according to format=flowed spec, is an indicator to
>> remove the newline and append it to the previous line.
> 
> Right, but: Does "format-flowed" apply to quoted text at all? I doubt it,
> but I didn't check the relevant RFC (yet).
> 
> If it would apply to quoted text, this could lead to completely destroyed
> quotes such as...
> 
> -------------------------------
>> line1 > line2 > line3 > line4
> -------------------------------
> 
> ...given that lines 1-3 have trailing spaces.
> 
> How does Thunderbird (or any other sane MUA) handle such lines?
> 
>> Fixed in https://github.com/roundcube/roundcubemail/commit/6d41d8fd4bbd.
>> I hope that doesn't create any new side-effects. If it does, please
>> report.
> 
> Sure, trust me. ;) (And thanks again)
> 
> Cheers,
> -- 
> Michael Heydekamp
> Co-Admin freexp.de
> Düsseldorf/Germany
> _______________________________________________
> Roundcube Users mailing list
> [email protected]
> http://lists.roundcube.net/mailman/listinfo/users

_______________________________________________
Roundcube Users mailing list
[email protected]
http://lists.roundcube.net/mailman/listinfo/users

Reply via email to