Remove useless Equatable constraint

[ptoffy]

We can get rid of the Equatable requirement in the multiparty body type typealias since it was really just used in tests

[gwynne]

Suggested change

	defer { self.base = nil } // release the reference so no CoW is triggered
	defer { self.base = nil } // release the reference so no CoW is triggered

[ptoffy]

This is swift-format's doing

[gwynne]

... That's bizarre, wrong, stupid, pointless, and so forth. Oh well.

[github-actions]

Benchmark Report

✅ Pull request has no significant performance differences ✅

Click to expand comparison result

Benchmark check running at 2025-01-16 11:37:12 UTC

The baseline 106/merge is EQUAL to the defined thresholds.

Click to expand benchmark result

Baseline 106/merge

Host a7f46776b36b with 2 aarch64 processors with 3 GB memory, running:
#23-Ubuntu SMP Mon Dec  9 23:51:16 UTC 2024

Parser

CollatingParserAllocations_256MiB

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) (K) *	82	82	82	82	82	82	82	1

CollatingParserAllocations_Empty

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) *	8	8	8	8	8	8	8	1

CollatingParserCPUTime_256MiB

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Time (user CPU) (ms) *	540	560	560	570	570	590	590	20

StreamingParserAllocations_256MiB

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) (K) *	82	82	82	82	82	82	82	1

StreamingParserAllocations_Empty

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) *	6	6	6	6	6	6	6	1

StreamingParserCPUTime_256MiB

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Time (user CPU) (ms) *	520	520	530	530	530	530	530	20

Serializer

100xSerializerCPUTime_1024Parts

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Time (user CPU) (ms) *	220	220	220	220	220	230	230	20

SerializerAllocations_1024Parts

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) *	3087	3087	3087	3087	3087	3087	3087	1

SerializerAllocations_Empty

Metric	p0	p25	p50	p75	p90	p99	p100	Samples
Malloc (total) *	4	4	4	4	4	4	4	1