you commented on line 166 ```if (!error_code.all())``` but that line has been removed in a later commit and is not in the "final" code of the PR. I suppose you looked at the commits individually instead of looking at the final changes? 
Or am I misunderstanding something here?