Yea, I agree with @wescarr, the general point still stands that `isReader` seems unnecessary and can be refactored away...