Use brackets. Return on own line.

awslabs · Aug 2, 2024 · d8b2a92 · d8b2a92
1 parent 7d40de7
commit d8b2a92
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/AWS.Messaging.Lambda/Services/DefaultLambdaMessageProcessor.cs b/src/AWS.Messaging.Lambda/Services/DefaultLambdaMessageProcessor.cs
@@ -264,11 +264,15 @@ public async Task DeleteMessagesAsync(IEnumerable<MessageEnvelope> messages, Can
  /// </summary>
  private async Task ResetVisibilityTimeoutForFailures()
  {
- if (_configuration.SQSEvent == null) return;
- if (!_configuration.UseBatchResponse) return;
- if (!_configuration.VisibilityTimeoutForBatchFailures.HasValue) return;
+ if (_configuration.SQSEvent == null || !_configuration.UseBatchResponse || !_configuration.VisibilityTimeoutForBatchFailures.HasValue)
+ {
+ return;
+ }
  var failureCount = _sqsBatchResponse.BatchItemFailures?.Count ?? 0;
- if (failureCount == 0) return;
+ if (failureCount == 0)
+ {
+ return;
+ }
 
  var lookup = _configuration.SQSEvent.Records.ToDictionary(x => x.MessageId);
  var visibilityTimeout = _configuration.VisibilityTimeoutForBatchFailures.Value;