chore: linting configuration adjustments (#798)

* chore: linting configuration adjustments

* refactor: set AlignTrailingComments to true

---------

Co-authored-by: maciejmakowski2003 <maciej.makowski2608@gmail.com>

Author: maciejmakowski2003

packages/react-native-audio-api/.clang-format

@@ -1,43 +1,33 @@
 ---
+ColumnLimit: 80
 AccessModifierOffset: -1
 AlignAfterOpenBracket: AlwaysBreak
 AlignConsecutiveAssignments: false
 AlignConsecutiveDeclarations: false
-AlignEscapedNewlinesLeft: true
+AlignEscapedNewlines: DontAlign
 AlignOperands: false
-AlignTrailingComments: false
+AlignTrailingComments: true
 AllowAllParametersOfDeclarationOnNextLine: false
-AllowShortBlocksOnASingleLine: false
+AllowShortBlocksOnASingleLine: Empty
 AllowShortCaseLabelsOnASingleLine: false
+AllowShortCompoundRequirementOnASingleLine: true
+AllowShortEnumsOnASingleLine: true
 AllowShortFunctionsOnASingleLine: Empty
-AllowShortIfStatementsOnASingleLine: false
+AllowShortIfStatementsOnASingleLine: Never
+AllowShortLambdasOnASingleLine: Inline
 AllowShortLoopsOnASingleLine: false
-AlwaysBreakAfterReturnType: None
 AlwaysBreakBeforeMultilineStrings: true
 AlwaysBreakTemplateDeclarations: true
 BinPackArguments: false
 BinPackParameters: false
-BraceWrapping:
-  AfterClass: false
-  AfterControlStatement: false
-  AfterEnum: false
-  AfterFunction: false
-  AfterNamespace: false
-  AfterObjCDeclaration: false
-  AfterStruct: false
-  AfterUnion: false
-  BeforeCatch: false
-  BeforeElse: false
-  IndentBraces: false
 BreakBeforeBinaryOperators: None
 BreakBeforeBraces: Attach
 BreakBeforeTernaryOperators: true
-BreakConstructorInitializersBeforeComma: false
+BreakConstructorInitializers: BeforeColon
 BreakAfterJavaFieldAnnotations: false
 BreakStringLiterals: false
-ColumnLimit: 80
 CommentPragmas: '^ IWYU pragma:'
-ConstructorInitializerAllOnOneLineOrOnePerLine: true
+PackConstructorInitializers: NextLine
 ConstructorInitializerIndentWidth: 4
 ContinuationIndentWidth: 4
 Cpp11BracedListStyle: true
@@ -54,37 +44,36 @@ IncludeCategories:
 IndentCaseLabels: true
 IndentWidth: 2
 IndentWrappedFunctionNames: false
-KeepEmptyLinesAtTheStartOfBlocks: false
 MacroBlockBegin: ''
 MacroBlockEnd: ''
 MaxEmptyLinesToKeep: 1
 NamespaceIndentation: None
-ObjCBlockIndentWidth: 2
-ObjCSpaceAfterProperty: true
-ObjCSpaceBeforeProtocolList: true
 PenaltyBreakBeforeFirstCallParameter: 1
 PenaltyBreakComment: 300
 PenaltyBreakFirstLessLess: 120
 PenaltyBreakString: 1000
 PenaltyExcessCharacter: 1000000
 PenaltyReturnTypeOnItsOwnLine: 200
 PointerAlignment: Right
-ReflowComments: true
+ReflowComments: false
+RequiresExpressionIndentation: OuterScope
 SortIncludes: true
 SpaceAfterCStyleCast: false
 SpaceBeforeAssignmentOperators: true
 SpaceBeforeParens: ControlStatements
 SpaceInEmptyParentheses: false
 SpacesBeforeTrailingComments: 1
-SpacesInAngles: false
+SpacesInAngles: Never
 SpacesInContainerLiterals: true
 SpacesInCStyleCastParentheses: false
-SpacesInParentheses: false
 SpacesInSquareBrackets: false
-Standard: Cpp11
-TabWidth: 2
 UseTab: Never
 ---
+Language: Cpp
+Standard: c++20
+---
 Language: ObjC
-ColumnLimit: 120
 BreakBeforeBraces: WebKit
+ObjCBlockIndentWidth: 2
+ObjCSpaceAfterProperty: true
+ObjCSpaceBeforeProtocolList: true

packages/react-native-audio-api/common/cpp/audioapi/HostObjects/AudioContextHostObject.cpp

@@ -33,7 +33,9 @@ JSI_HOST_FUNCTION_IMPL(AudioContextHostObject, resume) {
   auto promise = promiseVendor_->createAsyncPromise(
       [audioContext = std::move(audioContext)]() {
         auto result = audioContext->resume();
-        return [result](jsi::Runtime &runtime) { return jsi::Value(result); };
+        return [result](jsi::Runtime &runtime) {
+          return jsi::Value(result);
+        };
       });
   return promise;
 }
@@ -43,7 +45,9 @@ JSI_HOST_FUNCTION_IMPL(AudioContextHostObject, suspend) {
   auto promise = promiseVendor_->createAsyncPromise(
       [audioContext = std::move(audioContext)]() {
         auto result = audioContext->suspend();
-        return [result](jsi::Runtime &runtime) { return jsi::Value(result); };
+        return [result](jsi::Runtime &runtime) {
+          return jsi::Value(result);
+        };
       });
 
   return promise;

packages/react-native-audio-api/common/cpp/audioapi/HostObjects/OfflineAudioContextHostObject.cpp

@@ -20,7 +20,9 @@ JSI_HOST_FUNCTION_IMPL(OfflineAudioContextHostObject, resume) {
   auto audioContext = std::static_pointer_cast<OfflineAudioContext>(context_);
   auto promise = promiseVendor_->createAsyncPromise([audioContext]() {
     audioContext->resume();
-    return [](jsi::Runtime &runtime) { return jsi::Value::undefined(); };
+    return [](jsi::Runtime &runtime) {
+      return jsi::Value::undefined();
+    };
   });
 
   return promise;

packages/react-native-audio-api/common/cpp/audioapi/dsp/VectorMath.cpp

@@ -508,15 +508,15 @@ void multiply(
   bool source2Aligned = is16ByteAligned(inputVector2);
   bool destAligned = is16ByteAligned(outputVector);
 
-#define SSE2_MULT(loadInstr, storeInstr)           \
-  while (outputVector < endP) {                    \
-    pSource1 = _mm_load_ps(inputVector1);          \
+#define SSE2_MULT(loadInstr, storeInstr) \
+  while (outputVector < endP) { \
+    pSource1 = _mm_load_ps(inputVector1); \
     pSource2 = _mm_##loadInstr##_ps(inputVector2); \
-    dest = _mm_mul_ps(pSource1, pSource2);         \
-    _mm_##storeInstr##_ps(outputVector, dest);     \
-    inputVector1 += 4;                             \
-    inputVector2 += 4;                             \
-    outputVector += 4;                             \
+    dest = _mm_mul_ps(pSource1, pSource2); \
+    _mm_##storeInstr##_ps(outputVector, dest); \
+    inputVector1 += 4; \
+    inputVector2 += 4; \
+    outputVector += 4; \
   }
 
   if (source2Aligned && destAligned) // Both aligned.
@@ -647,15 +647,15 @@ void multiplyByScalarThenAddToOutput(
 
   bool destAligned = is16ByteAligned(outputVector);
 
-#define SSE2_MULT_ADD(loadInstr, storeInstr)   \
-  while (outputVector < endP) {                \
-    pSource = _mm_load_ps(inputVector);        \
-    temp = _mm_mul_ps(pSource, mScale);        \
+#define SSE2_MULT_ADD(loadInstr, storeInstr) \
+  while (outputVector < endP) { \
+    pSource = _mm_load_ps(inputVector); \
+    temp = _mm_mul_ps(pSource, mScale); \
     dest = _mm_##loadInstr##_ps(outputVector); \
-    dest = _mm_add_ps(dest, temp);             \
+    dest = _mm_add_ps(dest, temp); \
     _mm_##storeInstr##_ps(outputVector, dest); \
-    inputVector += 4;                          \
-    outputVector += 4;                         \
+    inputVector += 4; \
+    outputVector += 4; \
   }
 
   if (destAligned)

packages/react-native-audio-api/common/cpp/test/src/biquad/BiquadFilterTest.cpp

@@ -89,18 +89,18 @@ INSTANTIATE_TEST_SUITE_P(
     Frequencies,
     BiquadFilterFrequencyTest,
     ::testing::Values(
-        0.0f, // 0 Hz - the filter should block all input signal
-        10.0f, // very low frequency
+        0.0f,   // 0 Hz - the filter should block all input signal
+        10.0f,  // very low frequency
         350.0f, // default
         nyquistFrequency - 0.0001f, // frequency near Nyquist
-        nyquistFrequency)); // maximal frequency
+        nyquistFrequency));         // maximal frequency
 
 INSTANTIATE_TEST_SUITE_P(
     QEdgeCases,
     BiquadFilterQTestLowpassHighpass,
     ::testing::Values(
-        -770.63678f, // min value for lowpass and highpass
-        0.0f, // default
+        -770.63678f,  // min value for lowpass and highpass
+        0.0f,         // default
         770.63678f)); // max value for lowpass and highpass
 
 INSTANTIATE_TEST_SUITE_P(

packages/react-native-audio-api/ios/audioapi/ios/AudioAPIModule.h

@@ -13,18 +13,20 @@
 @class AudioSessionManager;
 @class LockScreenManager;
 
-@interface AudioAPIModule : RCTEventEmitter
+@interface AudioAPIModule
+    : RCTEventEmitter
 #ifdef RCT_NEW_ARCH_ENABLED
-                            <NativeAudioAPIModuleSpec, RCTCallInvokerModule, RCTInvalidating>
+      <NativeAudioAPIModuleSpec, RCTCallInvokerModule, RCTInvalidating>
 #else
-                            <RCTBridgeModule>
+      <RCTBridgeModule>
 #endif // RCT_NEW_ARCH_ENABLED
 
 @property (nonatomic, strong) AudioEngine *audioEngine;
 @property (nonatomic, strong) NotificationManager *notificationManager;
 @property (nonatomic, strong) AudioSessionManager *audioSessionManager;
 @property (nonatomic, strong) LockScreenManager *lockScreenManager;
 
-- (void)invokeHandlerWithEventName:(NSString *)eventName eventBody:(NSDictionary *)eventBody;
+- (void)invokeHandlerWithEventName:(NSString *)eventName
+                         eventBody:(NSDictionary *)eventBody;
 
 @end