Consider removing dependency on zend-loader

[michalbundyra]

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.

---

Originally posted by @acelaya at zendframework/zend-mail#185

[michalbundyra]

LGTM: kill it 👍

On 15 Dec 2017 23:42, "Alejandro Celaya" [email protected] 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
zendframework/zend-mail#185, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAJakA0Io3UFHVl5vZQRH2NYFXA4I7aHks5tAvXCgaJpZM4REFdX
.

---

Originally posted by @Ocramius at zendframework/zend-mail#185 (comment)

[samsonasik]

I've created PR #95 for it.