Update clang format to match rippled/xrpld (#17)

* As of rippled/xrpld 3.0.0

Author: ximinez

.clang-format

@@ -1,5 +1,21 @@
 ---
-Language:        Cpp
+BreakBeforeBraces: Custom
+BraceWrapping:
+  AfterClass: true
+  AfterControlStatement: true
+  AfterEnum: false
+  AfterFunction: true
+  AfterNamespace: false
+  AfterObjCDeclaration: true
+  AfterStruct: true
+  AfterUnion: true
+  BeforeCatch: true
+  BeforeElse: true
+  IndentBraces: false
+KeepEmptyLinesAtTheStartOfBlocks: false
+MaxEmptyLinesToKeep: 1
+---
+Language: Cpp
 AccessModifierOffset: -4
 AlignAfterOpenBracket: AlwaysBreak
 AlignConsecutiveAssignments: false
@@ -18,48 +34,43 @@ AlwaysBreakBeforeMultilineStrings: true
 AlwaysBreakTemplateDeclarations: true
 BinPackArguments: false
 BinPackParameters: false
-BraceWrapping:
-  AfterClass:      true
-  AfterControlStatement: true
-  AfterEnum:       false
-  AfterFunction:   true
-  AfterNamespace:  false
-  AfterObjCDeclaration: true
-  AfterStruct:     true
-  AfterUnion:      true
-  BeforeCatch:     true
-  BeforeElse:      true
-  IndentBraces:    false
 BreakBeforeBinaryOperators: false
-BreakBeforeBraces: Custom
 BreakBeforeTernaryOperators: true
 BreakConstructorInitializersBeforeComma: true
-ColumnLimit:     80
-CommentPragmas:  '^ IWYU pragma:'
+ColumnLimit: 80
+CommentPragmas: "^ IWYU pragma:"
 ConstructorInitializerAllOnOneLineOrOnePerLine: true
 ConstructorInitializerIndentWidth: 4
 ContinuationIndentWidth: 4
 Cpp11BracedListStyle: true
 DerivePointerAlignment: false
-DisableFormat:   false
+DisableFormat: false
 ExperimentalAutoDetectBinPacking: false
-ForEachMacros:   [ Q_FOREACH,  BOOST_FOREACH ]
+ForEachMacros: [Q_FOREACH, BOOST_FOREACH]
+IncludeBlocks: Regroup
 IncludeCategories:
-  - Regex:           '^<(BeastConfig)'
-    Priority:        0
-  - Regex:           '^<(ripple)/'
-    Priority:        2
-  - Regex:           '^<(boost)/'
-    Priority:        3
-  - Regex:           '.*'
-    Priority:        4
-IncludeIsMainRegex: '$'
+  - Regex: "^<(test)/"
+    Priority: 0
+  - Regex: '^[^/]*\.h'
+    Priority: 1
+  - Regex: "^<(xrpld)/"
+    Priority: 2
+  - Regex: "^<(xrpl|ripple)/"
+    Priority: 3
+  - Regex: "^<(boost)/"
+    Priority: 4
+  - Regex: "^.*/"
+    Priority: 5
+  - Regex: '^.*\.h'
+    Priority: 6
+  - Regex: ".*"
+    Priority: 7
+IncludeIsMainRegex: "$"
 IndentCaseLabels: true
 IndentFunctionDeclarationAfterType: false
-IndentWidth:     4
+IndentRequiresClause: true
+IndentWidth: 4
 IndentWrappedFunctionNames: false
-KeepEmptyLinesAtTheStartOfBlocks: false
-MaxEmptyLinesToKeep: 1
 NamespaceIndentation: None
 ObjCSpaceAfterProperty: false
 ObjCSpaceBeforeProtocolList: false
@@ -70,18 +81,25 @@ PenaltyBreakString: 1000
 PenaltyExcessCharacter: 1000000
 PenaltyReturnTypeOnItsOwnLine: 200
 PointerAlignment: Left
-ReflowComments:  true
-SortIncludes:    true
+ReflowComments: true
+RequiresClausePosition: OwnLine
+SortIncludes: true
 SpaceAfterCStyleCast: false
 SpaceBeforeAssignmentOperators: true
 SpaceBeforeParens: ControlStatements
 SpaceInEmptyParentheses: false
 SpacesBeforeTrailingComments: 2
-SpacesInAngles:  false
+SpacesInAngles: false
 SpacesInContainerLiterals: true
 SpacesInCStyleCastParentheses: false
 SpacesInParentheses: false
 SpacesInSquareBrackets: false
-Standard:        Cpp11
-TabWidth:        8
-UseTab:          Never
+Standard: Cpp11
+TabWidth: 8
+UseTab: Never
+QualifierAlignment: Right
+---
+Language: Proto
+BasedOnStyle: Google
+ColumnLimit: 0
+IndentWidth: 2

src/OfflineTool.cpp

@@ -284,7 +284,7 @@ getStdin()
 
 int
 runCommand(
-    const std::string& command,
+    std::string const& command,
     std::vector<std::string> const& args,
     boost::filesystem::path const& keyFile,
     std::optional<std::string> const& keyType,
@@ -386,7 +386,7 @@ getEnvVar(char const* name)
 
 void
 printHelp(
-    const boost::program_options::options_description& desc,
+    boost::program_options::options_description const& desc,
     boost::filesystem::path const& defaultKeyfile)
 {
     static std::string const name = "ripple-offline";

src/OfflineTool.h

@@ -50,7 +50,7 @@ doCreateKeyfile(
 
 int
 runCommand(
-    const std::string& command,
+    std::string const& command,
     std::vector<std::string> const& args,
     boost::filesystem::path const& keyFile,
     std::optional<std::string> const& keyType,

src/RippleKey.cpp

@@ -25,7 +25,9 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <ripple/json/to_string.h>
 #include <ripple/protocol/Sign.h>
 #include <ripple/protocol/jss.h>
+
 #include <boost/filesystem.hpp>
+
 #include <fstream>
 
 namespace offline {

src/Serialize.cpp

@@ -28,7 +28,9 @@
 #include <ripple/protocol/ErrorCodes.h>
 #include <ripple/protocol/HashPrefix.h>
 #include <ripple/protocol/Sign.h>
+
 #include <boost/filesystem.hpp>
+
 #include <fstream>
 
 namespace offline {

src/Serialize.h

@@ -21,6 +21,7 @@
 #include <ripple/protocol/KeyType.h>
 #include <ripple/protocol/SecretKey.h>
 #include <ripple/protocol/st.h>
+
 #include <optional>
 
 namespace boost {

src/test/KeyFileGuard.h

@@ -19,7 +19,9 @@
 //==============================================================================
 
 #include <ripple/beast/unit_test.h>
+
 #include <boost/filesystem.hpp>
+
 #include <fstream>
 
 namespace offline {

src/test/OfflineTool_test.cpp

@@ -27,7 +27,9 @@
 
 #include <ripple/beast/unit_test.h>
 #include <ripple/protocol/SecretKey.h>
+
 #include <boost/format.hpp>
+
 #include <string>
 
 namespace offline {
@@ -552,10 +554,11 @@ class OfflineTool_test : public beast::unit_test::suite
                 auto const key = RippleKey::make_RippleKey(keyFile);
 
                 auto const known = boost::str(
-                    boost::format("New ripple key created in %s\n"
-                                  "Key type is %s, and account ID is %s\n"
-                                  "\nThis file should be stored securely and "
-                                  "not shared\n\n") %
+                    boost::format(
+                        "New ripple key created in %s\n"
+                        "Key type is %s, and account ID is %s\n"
+                        "\nThis file should be stored securely and "
+                        "not shared\n\n") %
                     keyFile.string() % to_string(key.keyType()) %
                     toBase58(calcAccountID(key.publicKey())));
 

src/test/RippleKey_test.cpp

@@ -103,7 +103,7 @@ class RippleKey_test : public beast::unit_test::suite
         path const keyFile = subdir / ".ripple" / "secret-key.txt";
 
         // Try some failure cases before writing the file
-        auto badFile = [&](const char* toWrite,
+        auto badFile = [&](char const* toWrite,
                            std::string const& expectedException) {
             path const badKeyFile = subdir / "bad-key.txt";
             if (toWrite)

src/test/Serialize_test.cpp

@@ -76,7 +76,7 @@ class Serialize_test : public beast::unit_test::suite
         {
             // Need a variable here instead of a literal to make
             // some compilers happy
-            const std::uint64_t mantissa = 5678900000000000LLU;
+            std::uint64_t const mantissa = 5678900000000000LLU;
             BEAST_EXPECT(
                 obj[sfSendMax] ==
                 STAmount(