[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 4faab10 01/33: Adjust whitespace for microcon
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 4faab10 01/33: Adjust whitespace for microconcinnity |
Date: |
Mon, 3 May 2021 08:15:50 -0400 (EDT) |
branch: master
commit 4faab1050b9c3f6a997a523fa52afe7bde80f5b3
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
Adjust whitespace for microconcinnity
One blank line below 'config.hpp' because it's so special.
Argument names indented uniformly.
Comments on '#include' line begin in column 41.
In '*_test.cpp' unit tests:
#include "pchfile.hpp"
[blank line]
#include "whatever_class_is_tested_here.hpp"
[blank line]
#include [other_headers_needed_here]
Surround the middle line with blank lines, to distinguish the header
that is to be tested from headers used by the tests. That emphasis
makes the purpose of the test clearer, and that order ensures that
the incidental headers that follow do not mask a defect in the
physical design of the tested header.
---
global_settings_test.cpp | 2 +-
ledger_pdf.hpp | 1 +
md5sum.hpp | 1 +
md5sum_test.cpp | 1 +
path.hpp | 2 +-
path_utility.hpp | 4 ++--
rate_table.hpp | 1 +
regex_test.cpp | 1 +
wx_test_case.hpp | 1 +
wx_test_output.hpp | 1 +
10 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/global_settings_test.cpp b/global_settings_test.cpp
index e900277..b3e0f63 100644
--- a/global_settings_test.cpp
+++ b/global_settings_test.cpp
@@ -22,8 +22,8 @@
#include "pchfile.hpp"
#include "global_settings.hpp"
-#include "path.hpp"
+#include "path.hpp"
#include "test_tools.hpp"
void test_directory_exceptions()
diff --git a/ledger_pdf.hpp b/ledger_pdf.hpp
index 005d855..9023fca 100644
--- a/ledger_pdf.hpp
+++ b/ledger_pdf.hpp
@@ -23,6 +23,7 @@
#define ledger_pdf_hpp
#include "config.hpp"
+
#include "path.hpp"
#include <string>
diff --git a/md5sum.hpp b/md5sum.hpp
index 16c4791..18ff24d 100644
--- a/md5sum.hpp
+++ b/md5sum.hpp
@@ -23,6 +23,7 @@
#define md5sum_hpp
#include "config.hpp"
+
#include "path.hpp"
#include <climits> // CHAR_BIT
diff --git a/md5sum_test.cpp b/md5sum_test.cpp
index 650ad03..6e1bb2d 100644
--- a/md5sum_test.cpp
+++ b/md5sum_test.cpp
@@ -22,6 +22,7 @@
#include "pchfile.hpp"
#include "md5sum.hpp"
+
#include "path.hpp"
#include "test_tools.hpp"
diff --git a/path.hpp b/path.hpp
index 38312fd..71f334d 100644
--- a/path.hpp
+++ b/path.hpp
@@ -28,7 +28,7 @@
#include <fstream>
#include <ostream>
#include <string>
-#include <system_error> // std::error_code
+#include <system_error> // std::error_code
namespace fs
{
diff --git a/path_utility.hpp b/path_utility.hpp
index 8291629..f2ab7f8 100644
--- a/path_utility.hpp
+++ b/path_utility.hpp
@@ -31,8 +31,8 @@
#include <string>
LMI_SO fs::path modify_directory
- (fs::path const& original_filepath
- ,fs::path const& supplied_directory
+ (fs::path const& original_filepath
+ ,fs::path const& supplied_directory
);
LMI_SO std::string orthodox_filename
diff --git a/rate_table.hpp b/rate_table.hpp
index 18948dc..b9e2ca4 100644
--- a/rate_table.hpp
+++ b/rate_table.hpp
@@ -23,6 +23,7 @@
#define rate_table_hpp
#include "config.hpp"
+
#include "path.hpp"
#include <cstddef> // size_t
diff --git a/regex_test.cpp b/regex_test.cpp
index f1df042..12b8378 100644
--- a/regex_test.cpp
+++ b/regex_test.cpp
@@ -22,6 +22,7 @@
#include "pchfile.hpp"
#include "boost_regex.hpp"
+
#include "contains.hpp"
#include "test_tools.hpp"
#include "timer.hpp"
diff --git a/wx_test_case.hpp b/wx_test_case.hpp
index fa494bf..ed4cb55 100644
--- a/wx_test_case.hpp
+++ b/wx_test_case.hpp
@@ -23,6 +23,7 @@
#define wx_test_case_hpp
#include "config.hpp"
+
#include "path.hpp"
/// Base class for the test case objects.
diff --git a/wx_test_output.hpp b/wx_test_output.hpp
index 7143fb5..a123d48 100644
--- a/wx_test_output.hpp
+++ b/wx_test_output.hpp
@@ -23,6 +23,7 @@
#define wx_test_output_hpp
#include "config.hpp"
+
#include "path.hpp"
/// Class helping to check for the expected output file existence.
- [lmi-commits] [lmi] master c8143ad 02/33: Include "path.hpp" even if it might not be needed today, (continued)
- [lmi-commits] [lmi] master c8143ad 02/33: Include "path.hpp" even if it might not be needed today, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 1ac62bc 13/33: Rewrite a unit test, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master cff71ea 11/33: Improve documentation, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master f767cd4 17/33: Fix defect introduced 20190527T1635Z: erroneous documentation, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 325e88a 12/33: Increase filename portability, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 271a1b8 09/33: Note a deferred boost-migration question, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 2025958 06/33: Names are stronger than comments, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master bb267b7 18/33: Reinstate some recently-removed documentation, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 6d7a545 26/33: Reinsert a required empty line, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master ea4515f 04/33: Further deboostify, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 4faab10 01/33: Adjust whitespace for microconcinnity,
Greg Chicares <=
- [lmi-commits] [lmi] master f4f4756 05/33: Resolve a marked defect [283], Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master c98c00d 10/33: Say "basename" rather than "leaf", Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 1a68606 23/33: Conserve 80-character maximum width, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master e33e392 22/33: Correct and reformat the modify_directory() documentation, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master fd44389 03/33: Include appropriate headers, and say why they're included, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master fdc0f5a 07/33: Deboostify even more, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 41274a8 14/33: Improve documentation, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master e4d2f31 08/33: Rename a function, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 2038991 15/33: Improve concinnity, Greg Chicares, 2021/05/03
- [lmi-commits] [lmi] master 9235f33 16/33: Avoid implicit conversion to fs::path, Greg Chicares, 2021/05/03