Remove whitespace before closing parens

Author: jwakely

src/lists.cpp

@@ -529,7 +529,7 @@ struct list_issues {
 
 auto operator<<( std::ostream & out, list_issues const & x) -> std::ostream & {
    auto list_separator = "";
-   for (auto number : x.issues ) {
+   for (auto number : x.issues) {
       out << list_separator << "<iref ref=\"" << number << "\"/>";
       list_separator = ", ";
    }
@@ -565,14 +565,14 @@ auto operator<<( std::ostream & out, discover_new_issues const & x) -> std::ostr
    };
 
    std::map<std::string, std::vector<int>, status_order> added_issues;
-   for (auto const & i : new_issues ) {
+   for (auto const & i : new_issues) {
       auto j = std::lower_bound(old_issues.cbegin(), old_issues.cend(), std::get<0>(i), find_num{});
       if(j == old_issues.end() or std::get<0>(*j) != std::get<0>(i)) {
          added_issues[std::get<1>(i)].push_back(std::get<0>(i));
       }
    }
 
-   for (auto const & i : added_issues ) {
+   for (auto const & i : added_issues) {
       auto const item_count = std::get<1>(i).size();
       if(1 == item_count) {
          out << "<li>Added the following " << std::get<0>(i) << " issue: <iref ref=\"" << std::get<1>(i).front() << "\"/>.</li>\n";
@@ -612,14 +612,14 @@ auto operator << (std::ostream & out, discover_changed_issues x) -> std::ostream
    };
 
    std::map<std::tuple<std::string, std::string>, std::vector<int>, status_transition_order> changed_issues;
-   for (auto const & i : new_issues ) {
+   for (auto const & i : new_issues) {
       auto j = std::lower_bound(old_issues.begin(), old_issues.end(), std::get<0>(i), find_num{});
       if (j != old_issues.end()  and  std::get<0>(i) == std::get<0>(*j)  and  std::get<1>(*j) != std::get<1>(i)) {
          changed_issues[std::tuple<std::string, std::string>{std::get<1>(*j), std::get<1>(i)}].push_back(std::get<0>(i));
       }
    }
 
-   for (auto const & i : changed_issues ) {
+   for (auto const & i : changed_issues) {
       auto const item_count = std::get<1>(i).size();
       if(1 == item_count) {
          out << "<li>Changed the following issue to " << std::get<1>(std::get<0>(i))
@@ -749,7 +749,7 @@ int main(int argc, char* argv[]) {
       auto metadata = lwg::metadata::read_from_path(path);
 #if defined (DEBUG_LOGGING)
       // dump the contents of the section index
-      for (auto const & elem : metadata.section_db ) {
+      for (auto const & elem : metadata.section_db) {
          std::string temp = elem.first;
          temp.erase(temp.end()-1);
          temp.erase(temp.begin());

src/mailing_info.cpp

@@ -16,7 +16,7 @@ void replace_all_irefs(std::vector<lwg::issue> const & issues, std::string & s)
    // Format of an issue reference: <iref ref="ISS"/>
    // Format of anchor: <a href="lwg-INDEX.html#ISS">ISS</a>
 
-   for (auto i = s.find("<iref ref=\""); i != std::string::npos; i = s.find("<iref ref=\"") ) {
+   for (auto i = s.find("<iref ref=\""); i != std::string::npos; i = s.find("<iref ref=\"")) {
       auto j = s.find('>', i);
       if (j == std::string::npos) {
          throw std::runtime_error{"missing '>' after iref"};

src/report_generator.cpp

@@ -153,7 +153,7 @@ auto major_section(lwg::section_num const & sn) -> std::string {
    return out.str();
 }
 
-void print_date(std::ostream & out, gregorian::date const & mod_date ) {
+void print_date(std::ostream & out, gregorian::date const & mod_date) {
    out << mod_date.year() << '-';
    if (mod_date.month() < 10) { out << '0'; }
    out << mod_date.month() << '-';
@@ -162,9 +162,9 @@ void print_date(std::ostream & out, gregorian::date const & mod_date ) {
 }
 
 template<typename Container>
-void print_list(std::ostream & out, Container const & source, char const * separator ) {
+void print_list(std::ostream & out, Container const & source, char const * separator) {
    char const * sep{""};
-   for( auto const & x : source ) {
+   for (auto const & x : source) {
       out << sep << x;
       sep = separator;
    }
@@ -412,7 +412,7 @@ void print_issues(std::ostream & out, std::vector<lwg::issue> const & issues, lw
    issue_set_by_status    const  issues_by_status{ issues.begin(), issues.end() };
 
    issue_set_by_first_tag active_issues;
-   for (auto const & elem : issues ) {
+   for (auto const & elem : issues) {
       if (lwg::is_active(elem.stat)) {
          active_issues.insert(elem);
       }
@@ -430,7 +430,7 @@ void print_resolutions(std::ostream & out, std::vector<lwg::issue> const & issue
    // This construction calls out for filter-iterators
 //   std::multiset<lwg::issue, order_by_first_tag> pending_issues;
    std::vector<lwg::issue> pending_issues;
-   for (auto const & elem : issues ) {
+   for (auto const & elem : issues) {
       if (predicate(elem)) {
          pending_issues.emplace_back(elem);
       }
@@ -822,7 +822,7 @@ void report_generator::make_sort_by_section(std::vector<issue>& issues, fs::path
    }
    stable_sort(b, e, order_by_section{section_db});
    std::set<issue, order_by_major_section> mjr_section_open{order_by_major_section{section_db}};
-   for (auto const & elem : issues ) {
+   for (auto const & elem : issues) {
       if (is_active_not_ready(elem.stat)) {
          mjr_section_open.insert(elem);
       }
@@ -891,7 +891,7 @@ void report_generator::make_individual_issues(std::vector<issue> const & issues,
    issue_set_by_status    const  issues_by_status{ issues.begin(), issues.end() };
 
    issue_set_by_first_tag active_issues;
-   for (auto const & elem : issues ) {
+   for (auto const & elem : issues) {
       if (lwg::is_active(elem.stat)) {
          active_issues.insert(elem);
       }

src/sections.cpp

@@ -97,7 +97,7 @@ auto lwg::operator << (std::ostream& os, section_num const & sn) -> std::ostream
 //   if (!sn.prefix.empty()) { os << sn.prefix << " "; }
 
    bool use_period{false};
-   for (auto sub : sn.num ) {
+   for (auto sub : sn.num) {
       if (std::exchange(use_period, true)) {
          os << '.';
       }

src/toc_diff.cpp

@@ -15,7 +15,7 @@ namespace fs = std::filesystem;
 
 // DEBUG VISUALIZATION TOOL ONLY
 void display_issues(std::vector<std::pair<int, std::string> > const & issues) {
-   for( auto const & x : issues ) {
+   for( auto const & x : issues) {
       std::cout << x.first << "\t" << x.second << '\n';
    }
    std::cout << '\n';
@@ -35,7 +35,7 @@ auto remove_pending(std::string stat) -> std::string {
 
 auto remove_tentatively(std::string stat) -> std::string {
     typedef std::string::size_type size_type;
-    if( 0 == stat.find("Tentatively") ) {
+    if( 0 == stat.find("Tentatively")) {
         stat.erase(size_type{0}, size_type{12});
     }
     return stat;
@@ -48,7 +48,7 @@ auto remove_qualifier(std::string const & stat) -> std::string {
 
 
 auto find_file(std::string const & status) -> std::string {
-    if( 0 == status.find("Tentatively") ) {
+    if( 0 == status.find("Tentatively")) {
         return "lwg-active.html";
     }
 
@@ -150,7 +150,7 @@ struct list_issues {
 
 auto operator<<( std::ostream & out, list_issues const & x) -> std::ostream & {
    auto list_separator = "";
-   for (auto number : x.issues ) {
+   for (auto number : x.issues) {
       out << list_separator << "<iref ref=\"" << number << "\"/>";
       list_separator = ", ";
    }
@@ -176,14 +176,14 @@ auto operator<<( std::ostream & out, discover_new_issues const & x) -> std::ostr
    std::vector<std::pair<int, std::string> > const & new_issues = x.new_issues;
 
    std::map<std::string, std::vector<int> > added_issues;
-   for( auto const & i : new_issues ) {
+   for( auto const & i : new_issues) {
       auto j = std::lower_bound(old_issues.cbegin(), old_issues.cend(), i.first, find_num{});
       if(j == old_issues.end()) {
          added_issues[i.second].push_back(i.first);
       }
    }
 
-   for( auto const & i : added_issues ) {
+   for( auto const & i : added_issues) {
       auto const item_count = i.second.size();
       if(1 == item_count) {
          out << "<li>Added the following " << i.first << " issue: <iref ref=\"" << i.second.front() << "\"/>.</li>\n";
@@ -221,14 +221,14 @@ auto operator<<( std::ostream & out, discover_changed_issues x) -> std::ostream
    std::vector<std::pair<int, std::string> > const & new_issues = x.new_issues;
 
    std::map<std::pair<std::string, std::string>, std::vector<int>, reverse_pair> changed_issues;
-   for (auto const & i : new_issues ) {
+   for (auto const & i : new_issues) {
       auto j = std::lower_bound(old_issues.begin(), old_issues.end(), i.first, find_num{});
       if (j != old_issues.end()  and  i.first == j->first  and  j->second != i.second) {
          changed_issues[make_pair(j->second, i.second)].push_back(i.first);
       }
    }
 
-   for (auto const & i : changed_issues ) {
+   for (auto const & i : changed_issues) {
       auto const item_count = i.second.size();
       if(1 == item_count) {
          out << "<li>Changed the following issue from " << i.first.first << " to " << i.first.second