On May 16, 2013, at 9:56 AM, Alexey Melnichuk <mi...@newmail.ru> wrote:

> lua-pop3 v0.1.3

Nice. Thanks for sharing.

One question though…

The LIST [1] and UIDL [2] implementation seems to build a table of all the 
returned values, if I read the code right. This might not  fly for larger 
mailboxes though.

Consider:

STAT
+OK 632827 5248260641

Building a table to temporary hold these 632,827 messages data points is a bit 
of an overhead, no?

Have you perhaps consider turning these methods into proper iterators? 


[1] https://github.com/moteus/lua-pop3/blob/master/lua/pop3.lua#L532
[2] https://github.com/moteus/lua-pop3/blob/master/lua/pop3.lua#L565
------------------------------------------------------------------------------
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
_______________________________________________
Luarocks-developers mailing list
Luarocks-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/luarocks-developers

Reply via email to