Adjust some whitespace in 'if' and 'for' statements

Author: jwakely

src/lists.cpp

@@ -564,14 +564,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) {
       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)) {
+      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) {
       auto const item_count = std::get<1>(i).size();
-      if(1 == item_count) {
+      if (1 == item_count) {
          out << "<li>Added the following " << std::get<0>(i) << " issue: <iref ref=\"" << std::get<1>(i).front() << "\"/>.</li>\n";
       }
       else {
@@ -618,7 +618,7 @@ auto operator << (std::ostream & out, discover_changed_issues x) -> std::ostream
 
    for (auto const & i : changed_issues) {
       auto const item_count = std::get<1>(i).size();
-      if(1 == item_count) {
+      if (1 == item_count) {
          out << "<li>Changed the following issue to " << std::get<1>(std::get<0>(i))
              << " (from " << std::get<0>(std::get<0>(i)) << "): <iref ref=\"" << std::get<1>(i).front() << "\"/>.</li>\n";
       }

src/report_generator.cpp

@@ -324,7 +324,7 @@ void print_issue(std::ostream & out, lwg::issue const & iss, lwg::section_map &
 
          // When printing for the list, also emit an absolute link to the individual file.
          // Absolute link so that copying only the big lists elsewhere doesn't result in broken links.
-         if(type == print_issue_type::in_list) {
+         if (type == print_issue_type::in_list) {
               out << "<h3 id=\"" << iss.num << "\"><a href=\"#" << iss.num << "\">" << iss.num << "</a>";
               out << "<sup><a href=\"https://cplusplus.github.io/LWG/issue" << iss.num << "\">(i)</a></sup>";
          }
@@ -782,7 +782,7 @@ void report_generator::make_sort_by_section(std::span<issue> issues, fs::path co
    };
    std::ranges::sort(issues, {}, proj);
 
-   if(active_only) {
+   if (active_only) {
       auto status_priority = [](const issue& i) { return lwg::get_status_priority(i.stat); };
       // Find the first issue not in Voting, Immediate, or Ready status:
       auto first = std::ranges::upper_bound(issues, lwg::get_status_priority("Ready"), {}, status_priority);
@@ -811,7 +811,7 @@ void report_generator::make_sort_by_section(std::span<issue> issues, fs::path co
    out << "<h1>Index by Section</h1>\n";
    out << "<p>Reference " << is14882_docno << "</p>\n";
    out << "<p>This document is the Index by Section for the <a href=\"lwg-active.html\">Library Active Issues List</a>";
-   if(!active_only) {
+   if (!active_only) {
       out << ", <a href=\"lwg-defects.html\">Library Defect Reports and Accepted Issues</a>, and <a href=\"lwg-closed.html\">Library Closed Issues List</a>";
    }
    out << ".</p>\n";

src/section_data.cpp

@@ -107,7 +107,7 @@ replace_all(std::string s, const std::string& old, const std::string& nw)
 int main (int argc, char** argv)
 {
     std::string prefix;
-    if(argc > 1)
+    if (argc > 1)
         prefix = std::string(argv[1]);
 
     std::vector<std::pair<section_num, section_tag>> v;
@@ -121,7 +121,7 @@ int main (int argc, char** argv)
         std::cin >> n;
         if (std::cin.fail())
             throw std::runtime_error("incomplete tag / num pair");
-        if(!prefix.empty())
+        if (!prefix.empty())
             n.prefix = prefix;
 
         t = replace_all(t, "&", "&amp;");

src/set_status.cpp

@@ -109,10 +109,10 @@ int main(int argc, char const * argv[]) {
       auto old_status = issue_data.substr(k, l-k);
       issue_data.replace(k, l-k, new_status);
 
-      if(lwg::filename_for_status(new_status) != lwg::filename_for_status(old_status)) {
+      if (lwg::filename_for_status(new_status) != lwg::filename_for_status(old_status)) {
          // when performing a major status change, record the date and change as a note
          auto eod = issue_data.find("</discussion>");
-         if(eod == std::string::npos) {
+         if (eod == std::string::npos) {
              throw bad_issue_file{filename, "Unable to find end of discussion"};
          }
          std::ostringstream note;

src/status.cpp

@@ -18,7 +18,7 @@ constexpr std::string_view LWG_DEFECTS{"lwg-defects.html"};
 
 auto lwg::filename_for_status(std::string_view stat) -> std::string_view {
    // Tentative issues are always active
-   if(is_tentative(stat)) {
+   if (is_tentative(stat)) {
       return LWG_ACTIVE;
    }
 
@@ -78,21 +78,21 @@ auto lwg::is_tentative(std::string_view stat) -> bool {
 }
 
 auto lwg::is_assigned_to_another_group(std::string_view stat) -> bool {
-   for( auto s : {"Core", "EWG", "LEWG", "SG1", "SG9", "SG16" }) {
-     if(s == stat) return true;
+   for (auto s : {"Core", "EWG", "LEWG", "SG1", "SG9", "SG16" }) {
+     if (s == stat) return true;
    }
    return false;
 }
 
 auto lwg::is_not_resolved(std::string_view stat) -> bool {
    if (is_assigned_to_another_group(stat)) return true;
-   for( auto s : {"Deferred", "New", "Open", "Review"}) { if(s == stat) return true; }
+   for (auto s : {"Deferred", "New", "Open", "Review"}) { if (s == stat) return true; }
    return false;
 }
 
 auto lwg::is_votable(std::string_view stat) -> bool {
    stat = remove_tentatively(stat);
-   for( auto s : {"Immediate", "Voting"}) { if(s == stat) return true; }
+   for (auto s : {"Immediate", "Voting"}) { if (s == stat) return true; }
    return false;
 }
 
@@ -173,7 +173,7 @@ auto lwg::get_status_priority(std::string_view stat) noexcept -> std::ptrdiff_t
    auto const i = std::ranges::find(status_priority, stat);
 #if !defined(DEBUG_SUPPORT)
    // Diagnose when unknown status strings are passed
-   if(std::end(status_priority) == i) {
+   if (std::end(status_priority) == i) {
       std::cout << "Unknown status: " << stat << std::endl;
    }
 #endif