Comment 1 for bug 1746116

code is wrong. All of the intent for synchronous_reader throughout enginefacade as well as in the test code in test_enginefacade needs to have the role of synchronous_reader reversed. it is written as though it means to say "asynchronous_reader", which might have been a better name for this field.