From 6fc116012bd5eb922f7fc58a8800ec681908a106 Mon Sep 17 00:00:00 2001
From: David Anderson <davea@ssl.berkeley.edu>
Date: Sat, 22 Jul 2017 00:15:40 -0700
Subject: [PATCH] client: code cleanup, no functional change

---
 client/acct_mgr.cpp | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/client/acct_mgr.cpp b/client/acct_mgr.cpp
index 698083367a..402368841d 100644
--- a/client/acct_mgr.cpp
+++ b/client/acct_mgr.cpp
@@ -54,8 +54,7 @@ static const char *run_mode_name[] = {"", "always", "auto", "never"};
 // if URL is null, detach from current account manager
 //
 int ACCT_MGR_OP::do_rpc(
-    string _url, string name, string password_hash,
-    bool _via_gui
+    string _url, string name, string password_hash, bool _via_gui
 ) {
     int retval;
     unsigned int i;
@@ -246,7 +245,7 @@ void AM_ACCOUNT::handle_no_rsc(const char* name, bool value) {
     no_rsc[i] = value;
 }
 
-// parse account from AM reply
+// parse a project account from AM reply
 //
 int AM_ACCOUNT::parse(XML_PARSER& xp) {
     char buf[256];
@@ -745,9 +744,7 @@ void ACCT_MGR_OP::handle_reply(int http_op_retval) {
 }
 
 // write AM info to files.
-// This is done after each AM RPC,
-// perhaps overkill since the info doesn't generally change.
-// But doesn't matter since infrequent.
+// This is done after each AM RPC.
 //
 int ACCT_MGR_INFO::write_info() {
     FILE* f;
@@ -961,7 +958,9 @@ int ACCT_MGR_INFO::init() {
 
 bool ACCT_MGR_INFO::poll() {
     if (!using_am()) return false;
-    if (gstate.acct_mgr_op.gui_http->is_busy()) return false;
+    if (gstate.acct_mgr_op.gui_http->is_busy()) {
+        return false;
+    }
 
     if (gstate.now > next_rpc_time) {