cddb client protocol

[rhymes]

cddb client protocol implementation

[teratorn]

I had a quick read through your code and it looks pretty good. The doc strings 
are refreshingly well done :)

One question I have right now,

Have you considered writing a client interface that uses deferreds?
I understand that this is just a protocol implementation, but to
be most useful, users are going to want to integrate with their
existing deferred-using code.

I suggest you rename the CDDBPClient to the more verbose name 
CDDBClientProtocol. Then provide a high-level CDDBClient which one would use to 
connect to a cddb server and run queries. Appropriate methods should return 
deferrds, and the implementation would gather the results as they are available 
and fire the deferred once a full response has been received.

What do you think?

Anyway, it's late and I'm heading to bed.. maybe I'll think of something else 
tomorrow :)

[thomasvs]

didn't mean to change component to conch, trac did that automatically since it has to be from the dropdown.

[itamar]

I think this is a specialized enough protocol that we're not going to include it in Twisted proper. A standalone package ("txCddb", say) is probably the way to go.