-fix some clang warnings

src/benchmark/taler-aggregator-benchmark.c

@@ -504,7 +504,7 @@ run (void *cls,
     {
       struct GNUNET_HashCode seed;
       struct TALER_AgeMask mask = {
-        .bits = 1 || 1 << 8 || 1 << 12 || 1 << 16 || 1 << 18
+        .bits = 1 | (1 << 8) | (1 << 12) | (1 << 16) | (1 << 18)
       };
       struct TALER_AgeCommitmentProof acp = {0};
 

src/exchange/taler-exchange-httpd_batch-withdraw.c

@@ -173,7 +173,6 @@ batch_withdraw_transaction (void *cls,
                             MHD_RESULT *mhd_ret)
 {
   struct BatchWithdrawContext *wc = cls;
-  struct GNUNET_TIME_Timestamp now;
   uint64_t ruuid;
   enum GNUNET_DB_QueryStatus qs;
   bool balance_ok = false;
@@ -213,7 +212,7 @@ batch_withdraw_transaction (void *cls,
   }
   wc->kyc.ok = true;
   qs = TEH_plugin->do_batch_withdraw (TEH_plugin->cls,
-                                      now,
+                                      wc->now,
                                       wc->reserve_pub,
                                       &wc->batch_total,
                                       &found,
@@ -262,7 +261,7 @@ batch_withdraw_transaction (void *cls,
     qs = TEH_plugin->do_batch_withdraw_insert (TEH_plugin->cls,
                                                nonce,
                                                &pc->collectable,
-                                               now,
+                                               wc->now,
                                                ruuid,
                                                &denom_unknown,
                                                &conflict,

src/exchangedb/plugin_exchangedb_postgres.c

@@ -10210,6 +10210,7 @@ reserve_expired_cb (void *cls,
       ret = GNUNET_SYSERR;
       break;
     }
+    // FIXME: ret here is a qs! Bad enum conversion! FIX DESIGN!!!
     ret = erc->rec (erc->rec_cls,
                     &reserve_pub,
                     &remaining_balance,
@@ -15300,6 +15301,7 @@ postgres_delete_shard_locks (void *cls)
     GNUNET_PQ_EXECUTE_STATEMENT_END
   };
 
+  // FIXME: rval is a GNUNET_GenericReturnValue, not DB status!
   return GNUNET_PQ_exec_statements (pg->conn,
                                     es);
 }

src/mhd/mhd_parsing.c

@@ -60,7 +60,7 @@ TALER_MHD_parse_post_json (struct MHD_Connection *connection,
     GNUNET_break (NULL == *json);
     GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
                 "Closing connection, upload too large\n");
-    return MHD_NO;
+    return GNUNET_SYSERR;
   case GNUNET_JSON_PR_JSON_INVALID:
     GNUNET_break (NULL == *json);
     return (MHD_YES ==