chore: misc formatting updates and fixes (#7049)

* chore: remove redundant `.clang-format` entry for `TR_DEFINE_QUARK`

Left-over from bf41e1487a.

* chore: remove redundant `.clang-format` entry for `API_HEADER_*`

Left-over from f7edcfcb2a.

* chore: drop deprecated options in favour of `PackConstructorInitializers`

* fix: invalid `SpacesInAngles` value

* chore: add comment to forecast a change we need to make in the future

* chore: add trailing comma to prettify lambda

.clang-format

@@ -35,7 +35,7 @@ ReflowComments: false
 SortIncludes: Never
 SpaceAfterCStyleCast: false
 SpacesBeforeTrailingComments: 1
-SpacesInAngles: false
+SpacesInAngles: Never
 SpacesInCStyleCastParentheses: false
 SpacesInParentheses: false
 SpacesInSquareBrackets: false
@@ -46,24 +46,18 @@ Language: Cpp
 Standard: c++17
 
 AccessModifierOffset: -4
-AllowAllConstructorInitializersOnNextLine: false
+PackConstructorInitializers: Never
 AlwaysBreakAfterReturnType: None
 AlwaysBreakTemplateDeclarations: Yes
 BreakConstructorInitializers: BeforeComma
 BreakInheritanceList: BeforeComma
-ConstructorInitializerAllOnOneLineOrOnePerLine: false
 ConstructorInitializerIndentWidth: 4
 FixNamespaceComments: true
 IndentGotoLabels: false
 KeepEmptyLinesAtTheStartOfBlocks: false
 QualifierAlignment: Right
-SortUsingDeclarations: true
+SortUsingDeclarations: true # Since clang-format 16, the equivalent value is LexicographicNumeric
 SpaceAfterTemplateKeyword: false
-StatementMacros:
-- TR_DEFINE_QUARK
-WhitespaceSensitiveMacros:
-- API_HEADER
-- API_HEADER_CRYPT
 ---
 Language: ObjC
 

libtransmission/announcer-udp.cc

@@ -512,11 +512,10 @@ private:
                     fmt::arg("error_code", static_cast<int>(rc))));
             return {};
         }
-        auto const info_uniq = std::unique_ptr<addrinfo, void (*)(addrinfo*)>{ info,
-                                                                               [](addrinfo* p) // MSVC forced my hands
-                                                                               {
-                                                                                   freeaddrinfo(p);
-                                                                               } };
+        auto const info_uniq = std::unique_ptr<addrinfo, void (*)(addrinfo*)>{
+            info,
+            [](addrinfo* p) { freeaddrinfo(p); }, // MSVC forced me to write this lambda wrapper
+        };
 
         // N.B. getaddrinfo() will return IPv4-mapped addresses by default on macOS
         auto socket_address = tr_socket_address::from_sockaddr(info->ai_addr);