why this opening bracket here and the closing one in line 186? Apart from that LGTM