[Bug]: Ecc: Spurious alarm when typedef spans two lines

[deeglaze]

Contact Details

[email protected]

Describe the Bug

The DeclCheckSameStructure error has a false positive with

typedef _VERY_LONG_PROTOCOL_NAME
  VERY_LONG_PROTOCOL_NAME;

...

struct _VERY_LONG_PROTOCOL_NAME {
  int field0;
};

and can be "fixed" with an over-long line where the typedef without the line break is longer than 80 characters.

Reproduction steps

1. I was CI testing https://github.com/deeglaze/edk2/tree/enable_umv8
2. I had the initial typedef in MdePkg/Include/Protocol/MemoryAcceptance.h span two lines.

The CI alarmed saying that I had defined _BZ3987_MEMORY_ACCEPTANCE_PROTOCOL twice, which is not correct.

Expected behavior

The typedef with a line break should not trigger this alarm.

A typedef should be allowed to span two lines if it would otherwise take more than 80 characters.

Execution Environment

Ran in edk2's CI

Pip packages

No response

Additional context

No response