[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Review of sort
Mark Crispin wrote:
Cyrus -
On Wed, 15 Nov 2006, Cyrus Daboo wrote:
- In Section 1 it states that the server SHOULD support COMPARATOR if
the SORT extension is defined, but it does not explicitly state the
same for the THREAD extension. Since THREAD uses SEARCH and has a
subject matching step, I think it should also say SHOULD support
COMPARATOR.
Would it be alright if this was simply generalized to "SORT and/or
THREAD extension"?
I think so. I'd also suggest pulling that sentence out of the second
paragraph, and either appending it to the first paragraph or making it
new paragraph it at the end of Section 1.
- In Section 2.1 it states that the 'base subject' algorithm must be
used when 'sorting by subject'. It does not explicitly state that it
also needs to be used when doing THREAD=ORDEREDSUBJECT and
THREAD=REFERENCES - it should.
Would it be alright if this was generalized to "sorting or threading by
subject" (without having to specifically list threading algorithms)?
Sure. We could also change the first sentence in 2.1 to match.
- Currently the (expired) COMPARATOR extension states it applies to
THREAD=ORDEREDSUBJECT but not THREAD=REFERENCES. Yet the later
includes a base subject comparison step, so COMPARATOR should be used
for that too.
This is an issue in the COMPARATOR document, isn't it?
Yes.
--
Kenneth Murchison
Systems Programmer
Project Cyrus Developer/Maintainer
Carnegie Mellon University