go-imap icon indicating copy to clipboard operation
go-imap copied to clipboard

Code should use unidirectional channels, where appropriate

Open ThomasHabets opened this issue 2 years ago • 1 comments

The API docs would be easier to read if e.g. Client.Fetch took a write-only channel. Then it would be clear that it's an output.

ThomasHabets avatar Feb 17 '22 15:02 ThomasHabets

Indeed. Unfortunately this doesn't bode well with our (arguably bad) implementation in responses.Fetch, which is used for both the client and the server.

emersion avatar Feb 17 '22 16:02 emersion

Closing because go-imap v2 stops using Go channels in the public API.

emersion avatar Mar 23 '23 10:03 emersion