On Fri, 20 Nov 2015, Dan Fandrich wrote:

Commit 64e959ff removed most of the TODO comments in the code because "They
tend to never get updated anyway so they're frequently inaccurate and we never
go back to revisit them anyway." I disagree with this removal.

Okay, I hear you. I got annoyed by some that clearly were inaccurate/old and then it struck me that these comments tend to get added to the code and then never get updated. I probably overreacted when I removed all of them like that. Thanks for speaking up.

I'll revert that patch now and I'll instead address the particular ones I think are the most wrong to start with. In the long run I think we're better off adding issues to the TODO and KNOWN_BUGS documents so that they're easier to find.

--

 / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to