tfpt review “/shelveset:crit_argf;REDMOND\jdeville”
cleans up a few critical tags:
- re-patches mock.rb with the patch to remove respond_to last
- implements a basic implementation of ARGF, there are many holes that
still need to be filled here, but the basic idea works, and the specs
are no longer critical.
Could you add comments “TODO: thread safety” to the added fields. There
are several places that access them in thread unsafe way.
Could you also replace
// TODO: convert any non-string