Konstantin Varlamov

Results 2 issues of Konstantin Varlamov

Also fix a potential bug in `DescribeFailedFuture` (`error_message()` can return null).

api: firestore
cla: yes
tests: failed

The following variable appears to be unused: https://github.com/docopt/docopt.cpp/blob/400e6dd8e59196c914dcc2c56caf7dae7efa5eb3/docopt_private.h#L549 For context, I work on libc++ and we are considering enabling the unused variable warnings for some standard library types, including `std::string`....