NetTrace file format doc doesn't match implementation

[hoyosjs]

My guess is the implementation in trace event is actually right, but the spec says

perfview/src/TraceEvent/EventPipe/EventPipeFormat.md

Line 223 in edf42fe

- in either case, if MetadataId != 0 increment SequenceNumber by 1

Whereas the implementation only checks on metadata refs nullity in the case where the sequence flag isn't set:

perfview/src/TraceEvent/EventPipe/EventPipeEventSource.cs

Lines 1505 to 1517 in edf42fe

	if ((flags & (byte)CompressedHeaderFlags.CaptureThreadAndSequence) != 0)
	{
	header.SequenceNumber += (int)ReadVarUInt32(ref headerPtr) + 1;
	header.CaptureThreadId = (long)ReadVarUInt64(ref headerPtr);
	header.CaptureProcNumber = (int)ReadVarUInt32(ref headerPtr);
	}
	else
	{
	if(header.MetaDataId != 0)
	{
	header.SequenceNumber++;
	}
	}

Even though there's no guarantee that the sequence flag implies a non-mull metadata ID. Which one is correct? I assume the implementation is correct - otherwise the first event would have a negative seq number from what I've seen.