[Kde-pim] Review Request: Improve parsing of Content-ID header

[Torgny Nyblom]

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

(Updated 2010-01-05 09:10:07.999285)


Review request for KDE PIM.


Summary
-------

Add the parse() and identifier() functions to ContentID class.
The reason being that although Content-Id should obey they same rules as Message-Id (localpart at hostpart) in practice it doesn't.

The added parse() function tries to use the same algorithm as is used for MessageID but if that fails a debug message is issued and a more relaxed algorithm is used (parseAtom, might not be the best alternative but it works for the test email I've got).

The added identifier() function is also almost the same as in the base class (4 lines so not that big duplication). The difference being that if the relaxed parse() function was used to extract the header the domain part is not present and in that case the trailing '@' is discarded before returning,


Diffs (updated)
-----

  /branches/work/akonadi-ports/kdepimlibs/kmime/kmime_headers.h 1068282 
  /branches/work/akonadi-ports/kdepimlibs/kmime/kmime_headers.cpp 1068282 
  /branches/work/akonadi-ports/kdepimlibs/kmime/kmime_headers_p.h 1068282 
  /branches/work/akonadi-ports/kdepimlibs/kmime/tests/kmime_message_test.h 1068282 
  /branches/work/akonadi-ports/kdepimlibs/kmime/tests/kmime_message_test.cpp 1068282 

Diff: http://reviewboard.kde.org/r/2483/diff


Testing
-------

Old unittest works, new unittest for the added functions work.
KMail seem to get the expected values back.


Thanks,

Torgny

_______________________________________________
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/