[Kde-pim] Review Request 113750: Fix 286553 - Kmail signed emails invalid in Thunderbird w/ Enigmail

[Kevin Krammer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113750/#review43301
-----------------------------------------------------------



messagecomposer/job/signjob.cpp
<http://git.reviewboard.kde.org/r/113750/#comment31185>

    I know this has already been committed, but why all the bytearray -> string conversions if all following operations are again on QByteArrays?
    Seems like lots of needless from/to conversions


- Kevin Krammer


On Nov. 9, 2013, 5:36 p.m., Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113750/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2013, 5:36 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Bugs: 286553 and 303305
>     http://bugs.kde.org/show_bug.cgi?id=286553
>     http://bugs.kde.org/show_bug.cgi?id=303305
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> replace "From " and "--" at the beginning of lines with encoded versions according to RfC 3156, 3
> Note: If any line begins with the string "From ", it is strongly
>    suggested that either the Quoted-Printable or Base64 MIME encoding be applied.
> 
> 
> Diffs
> -----
> 
>   messagecomposer/job/signjob.cpp 0183963c9b484bef6eb5cf19668ed4bb5bdf84e8 
>   messagecomposer/tests/signjobtest.h d81ad0948a50f2e05283530e253c766d6cd99beb 
>   messagecomposer/tests/signjobtest.cpp ee300f2652fe645ed5dd3ba7824840ebce0f218f 
> 
> Diff: http://git.reviewboard.kde.org/r/113750/diff/
> 
> 
> Testing
> -------
> 
> * create a test for it.
> * test created message with Thunderbird.
> 
> 
> Thanks,
> 
> Sandro Knauß
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/