ezcMailPartParser ================= [x] The function parseHeader() should be refactored. The new code has in the if() and elseif() case duplicated code that can easily be extracted into a new private method. The code is too coupled and the private function will have 4-5 params. It applies to only 4 lines of code. The solution which I tried actually looks worse than what it is now. [x] The test in trunk/Mail/tests/composer_test.php should removed the added check for "charset=binary" as that is a bug in PHP (line 647ish) [ ] From reading the code, I don't think that it is possible to use headers with the same name while creating mail as well. This needs to be supported so that mails can be forwarded with all current headers in tact. We'd need to add tests for both generation and forwarding/replying here.