Consider removing dependency on zend-loader

[acelaya]

I have seen that this package requires zendframework/zend-loader, just for one class, the Zend\Mail\Header\HeaderLoader.

However, this class basically defines a map to resolve header names to class names.

Would you be open to replace this by a simple array map in the Zend\Mail\Headers class? This would let us drop the requirement on zendframework/zend-loader (assuming I'm not missing anything...).

I could do it if you want.

[Ocramius]

LGTM: kill it 👍

…

On 15 Dec 2017 23:42, "Alejandro Celaya" ***@***.***> wrote: I have seen that this package requires zendframework/zend-loader, just for one class, the Zend\Mail\Header\HeaderLoader. However, this class basically defines a map to resolve header names to class names. Would you be open to replace this by a simple array map in the Zend\Mail\Headers class? This would let us drop the requirement on zendframework/zend-loader (assuming I'm not missing anything...). I could do it if you want. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#185>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAJakA0Io3UFHVl5vZQRH2NYFXA4I7aHks5tAvXCgaJpZM4REFdX> .

[michalbundyra]

This repository has been closed and moved to laminas/laminas-mail; a new issue has been opened at laminas/laminas-mail#40.