Create specific exception instead of generic Exception

[lsoares]

try {
    req.await();
    // "other code"
} catch (Exception e) {
}

Dealing with generic exceptions creates some dangers. I propose that you create a specific exception (or more) like GeocodingException or likewise.

Otherwise, I may be be preventing runtime exceptions (and also catched exceptions) under "other code" to bubble up...

(I can also do it as PR if you prefer.)

[domesticmouse]

I'm always up for reviewing PRs. Thanks!

[pieteradejong]

the code fragment refers to the readme.
is the intention to update the readme,
or to modify (make more specific) (some of) the 9 occurrences thoughout the code?

[lsoares]

I meant in the code. I tried to do the PR but it bears lots of changes..

[pieteradejong]

@lsoares do you see a path toward breaking it up into smaller pieces?

[lsoares]

What if await under PendingResult throws an application exception (e.g. GoogleMapsException) instead of Exception? That would be a start I guess.

[domesticmouse]

I'm going to close this out for now. Please feel free to submit either the uber-PR or a series of smaller PRs. I'm about to break the world by migrating from OkHttp2 to OkHttp3, however...