Fix exception handling and operator precedence bugs (#295)

- Add return statement in catch(...) block to prevent fallthrough
- Fix operator precedence in GetState() to capture Call() return value

Fixes #201, #202

Co-authored-by: Levi Neely <lkn@darkstar.example.net>

main.cpp

@@ -214,6 +214,7 @@ int main(int argc, char **argv) {
     return 1;
   } catch (...) {
     std::cerr << "Exception of unknown type!" << std::endl;
+    return 1;
   }
 
   if (daemon) {

rpcclient.cpp

@@ -135,7 +135,7 @@ int RpcClient::GetState(pCloud_FileState *state, char *path) {
   size_t errm_size = 0;
   int rep = 0;
 
-  if ((rep = this->Call(4, path, &errm, &errm_size) == 0)) {
+  if ((rep = this->Call(4, path, &errm, &errm_size)) == 0) {
     pdbg_logf(D_NOTICE, "rpc_get_state responese rep[%d] path[%s]", rep, path);
     if (errm) {
       pdbg_logf(D_NOTICE, "The error is %s", errm);