Merge pull request #249 from Cysharp/hadashiA/fix-brotli-encoder

Fix BrotliCompressor.CopyToAsync buffer is not enough

Author: hadashiA

src/MemoryPack.Core/Compression/BrotliCompressor.cs

@@ -174,10 +174,18 @@ public async ValueTask CopyToAsync(Stream stream, int bufferSize = 65535, Cancel
             }
 
             // call BrotliEncoderOperation.Finish
-            var finalStatus = encoder.Compress(ReadOnlySpan<byte>.Empty, buffer, out var consumed, out var written, isFinalBlock: true);
-            if (written > 0)
+            var finalStatus = OperationStatus.DestinationTooSmall;
+            while (finalStatus == OperationStatus.DestinationTooSmall)
             {
-                await stream.WriteAsync(buffer.AsMemory(0, written), cancellationToken).ConfigureAwait(false);
+                finalStatus = encoder.Compress(ReadOnlySpan<byte>.Empty, buffer, out var consumed, out var written, isFinalBlock: true);
+                if (written > 0)
+                {
+                    await stream.WriteAsync(buffer.AsMemory(0, written), cancellationToken).ConfigureAwait(false);
+                }
+            }
+            if (finalStatus != OperationStatus.Done)
+            {
+                MemoryPackSerializationException.ThrowCompressionFailed(finalStatus);
             }
         }
         finally