Code review comment for lp://qastaging/~sylvain-calador/poweremail/header-non-ascii

Revision history for this message
Sylvain Calador (sylvain-calador) wrote :

Hi,

Great improvement from Jordi Esteve about internationalized header.

I just propose to surround the encoding stuff with try/catch to avoid
various exceptions when header is not internationalized but contains
non-ascii characters (I have sevral mail from Dell in this case, wich
give me errors like "ascii codec can't decode char at...")

Best regards,

Sylvain

« Back to merge proposal