[dev] Re: [cvs] commit: imp message.php imp/lib Message.php
Michael M Slusarz
slusarz@bigworm.colorado.edu
Thu, 20 Jun 2002 13:00:09 -0600
Quoting Michael M Slusarz <slusarz@bigworm.colorado.edu>:
| slusarz 2002/06/20 11:56:00 PDT
|
| Modified files:
| . message.php
| lib Message.php
| Log:
| Be much smarter about when to resort the mailbox index. Move the logic
| to
| IMP_Message:: so that it is better tied to the events that would
| trigger
| a resort.
This responds to some concerns/critiques Chuck was raising a few days ago
about the amount of re-sorts done while viewing messages. The re-sort
triggers are listed in the phpdoc above needResort() - i won't repeat them
here. Let me know if this looks good.
And now that I am looking at that method name, maybe i should change it to
needReSort(). i think we could all use a resort (vacation, that is).
Now on to tackling the indexing foolishness if new messages arrive while
moving between messages.
michael
______________________________________________
Michael Slusarz [slusarz@bigworm.colorado.edu]
The University of Colorado at Boulder