ILSpy

9645 Commits

54 Branches

94 Tags

91 MiB

Branch: fix/lambda-parameter-syntax

Commit Graph

Author	SHA1	Message	Date
Siegfried Pammer	fb75e42385	Assign a text location to comment-only empty statements Decompiler warnings (ILFunction.Warnings, e.g. the DetectPinnedRegions block-duplication notice) are surfaced as an EmptyStatement carrying only a comment. VisitEmptyStatement prints no semicolon for it, and EmptyStatement derives its StartLocation/EndLocation from its Location field, which is only set when that semicolon token is written. The statement was therefore left without a text location, and SequencePointBuilder then asserted on the empty start location while generating PDB sequence points - aborting PDB generation for any assembly whose decompilation emits such a warning (e.g. System.Net.Requests). Point the empty statement at the comment it carries (already printed by the time the node ends, so its location is known), falling back to the collapsed end-of-last-token position. Every printed node then has a location, so the SequencePointBuilder invariant holds without special-casing, and the statement lines up with the text it represents. Assisted-by: Claude:claude-opus-4-8:Claude Code	3 months ago

Author

SHA1

Message

Date

Siegfried Pammer

fb75e42385

Assign a text location to comment-only empty statements

Decompiler warnings (ILFunction.Warnings, e.g. the DetectPinnedRegions
block-duplication notice) are surfaced as an EmptyStatement carrying only
a comment. VisitEmptyStatement prints no semicolon for it, and
EmptyStatement derives its StartLocation/EndLocation from its Location
field, which is only set when that semicolon token is written. The
statement was therefore left without a text location, and
SequencePointBuilder then asserted on the empty start location while
generating PDB sequence points - aborting PDB generation for any assembly
whose decompilation emits such a warning (e.g. System.Net.Requests).

Point the empty statement at the comment it carries (already printed by
the time the node ends, so its location is known), falling back to the
collapsed end-of-last-token position. Every printed node then has a
location, so the SequencePointBuilder invariant holds without
special-casing, and the statement lines up with the text it represents.

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

3 months ago

1 Commits (fix/lambda-parameter-syntax)