Re: Re: [PEPr] Comment on Networking::Net_Cyrus

Zitat von "Damian Fernandez Sosa (DESC)" <damlists@cnba.uba.ar>:

Quoting Jan Schneider <jan@horde.org>:

Your package trades code reuse (Net_IMAP) with bloat (and thus probably performance hit) because users of your package would need to install, load and parse the huge Net_IMAP class while they only need a very small functionality from it.

Net_IMAP also implements DIGEST-MD5,CRAM-MD5, LOGIN and PLAIN auth methods (and automaticaly chooses the best method) so we can use those methods to add passwd's encryption to remote Net_Cyrus connections. Horde's Net_Cyrus doesn't Suppose you'll create an user or folder whith non-us characters (á,é ñ, etc) Horde's Net_Cyrus don't handle that situation! Net_Cyrus based on Net_IMAP does (because all that class's bloat) Also Suppose you have a newer version of cyrus (v. 2.1.12 or up) . It is not true that the folder deliniter will allways be '.' it can be '/' also (see unixhierarchysep's imapd.conf parammeter). Horde's Net_Cyrus don't handle this.

That convinces me.

Net_IMAP's size is about 48kb and IMAPProtocol.php's size is about 82kb (130kb total) Net_Cyrus can directly use IMAPProtocol.php and reduce about 48kb. and a lot of less methods so the parsing speed will increase a lot. Also I'm planning to split the Generic IMAP Parser from IMAPProtocol and directly use it in Net_IMAP and Net_Cyrus reducing the size to 60kb (50% decrement in size) and allowing to use only the methods needed. Again, the parsing and load speed can be increased

That would be cool.

Regarding Etienne's question on dev@lists.horde.org, this package is probably not necessary if you have a recent imap extension and c-client.

And a CVS's version of PHP 5 If this package is not necessary, also horde's Net_Cyrus isn't (it also does'nt use the newest c-client) You must have the newest version of c-client and compile it you can't use the php that comes in redhat, mandrake, suse, etc. I think debian has the support as a package in unstable . Also, that support is _ONLY_ in PHP 5 CVS. so you can only use c-client version in PHP 5.

That's not true. It works fine here with PHP 4.3.5 and c-client 2002d. Jan. -- http://www.horde.org - The Horde Project http://www.ammma.de - Neue Wege des Lernens http://www.tip4all.de - Deine private Tippgemeinschaft