ILSpy

9645 Commits

54 Branches

94 Tags

91 MiB

Branch: fix/lambda-parameter-syntax

Commit Graph

Author	SHA1	Message	Date
Siegfried Pammer	5fa44fab6a	Emit member-initializer sequence points in every constructor A field, auto-property, or event initializer is written once at its declaration, but in IL it runs in every instance constructor that does not chain to this(...) (and static initializers run in the static constructor). The decompiler lifts the initializer from a single constructor, so its breakpoint was emitted only there and the other constructors had none. Two causes are addressed: - The lift discarded the initializer's copies in the other constructors. They are now kept on MemberInitializerInOtherConstructorsAnnotation and replayed by SequencePointBuilder, mapping the same source location onto each constructor's IL. - PortablePdbWriter only emitted methods that DebugInfoGenerator discovered through declaration syntax, so a constructor whose declaration is omitted from the output (implicit default ctor, implicit static ctor, primary ctor) dropped its generated points. Those functions are now emitted by walking the sequence-point map directly. PdbGen fixtures cover single, multiple, this()-chained, implicit, static, primary-constructor, and field-like event initializers, pinning the reconstructed breakpoint map against the C# compiler's. Assisted-by: Claude:claude-opus-4-8:Claude Code	3 months ago
Siegfried Pammer	fe50c00f2f	Cover PdbGen loops and ordered residuals The PDB sequence-point tests were missing real while-loop input, and their residual comparison treated breakpoint locations as an unordered multiset. Add coverage for while/do-while fixtures and compare residuals in sequence order so stepping-order changes are pinned. Assisted-by: CodeAlta:gpt-5.5:CodeAlta	3 months ago
Siegfried Pammer	853220dba9	Cover more constructs in the PDB sequence-point tests Extends the breakpoint-map comparison to hidden sequence points, anchoring each hidden point to the visible point it follows so the descriptor stays independent of the IL offsets the decompiler reconstructs. Adds PdbGen cases spanning try/catch/finally, switch, async/await, yield, loops, LINQ, pattern matching and more, pinning the known residuals where the decompiler folds a compiler-hidden branch into an adjacent point. Assisted-by: Claude:claude-opus-4-8:Claude Code	3 months ago

Author

SHA1

Message

Date

Siegfried Pammer

5fa44fab6a

Emit member-initializer sequence points in every constructor

A field, auto-property, or event initializer is written once at its
declaration, but in IL it runs in every instance constructor that does not
chain to this(...) (and static initializers run in the static constructor).
The decompiler lifts the initializer from a single constructor, so its
breakpoint was emitted only there and the other constructors had none.

Two causes are addressed:

- The lift discarded the initializer's copies in the other constructors.
  They are now kept on MemberInitializerInOtherConstructorsAnnotation and
  replayed by SequencePointBuilder, mapping the same source location onto
  each constructor's IL.
- PortablePdbWriter only emitted methods that DebugInfoGenerator discovered
  through declaration syntax, so a constructor whose declaration is omitted
  from the output (implicit default ctor, implicit static ctor, primary
  ctor) dropped its generated points. Those functions are now emitted by
  walking the sequence-point map directly.

PdbGen fixtures cover single, multiple, this()-chained, implicit, static,
primary-constructor, and field-like event initializers, pinning the
reconstructed breakpoint map against the C# compiler's.

Assisted-by: Claude:claude-opus-4-8:Claude Code

3 months ago

Siegfried Pammer

fe50c00f2f

Cover PdbGen loops and ordered residuals

The PDB sequence-point tests were missing real while-loop input, and their residual comparison treated breakpoint locations as an unordered multiset. Add coverage for while/do-while fixtures and compare residuals in sequence order so stepping-order changes are pinned.

Assisted-by: CodeAlta:gpt-5.5:CodeAlta

3 months ago

Siegfried Pammer

853220dba9

Cover more constructs in the PDB sequence-point tests

Extends the breakpoint-map comparison to hidden sequence points, anchoring
each hidden point to the visible point it follows so the descriptor stays
independent of the IL offsets the decompiler reconstructs. Adds PdbGen cases
spanning try/catch/finally, switch, async/await, yield, loops, LINQ, pattern
matching and more, pinning the known residuals where the decompiler folds a
compiler-hidden branch into an adjacent point.

Assisted-by: Claude:claude-opus-4-8:Claude Code

3 months ago

3 Commits (fix/lambda-parameter-syntax)