pimhwang/ILSpy - ILSpy - git.quantfans.net

Author	SHA1	Message	Date
Siegfried Pammer	18d960e6b9	Say what the code does instead of which app did it first A comment that justifies behaviour by pointing at the WPF front-end means nothing to someone reading the file cold: the reason is either already stated beside it or is not stated anywhere. Each of these now names the constraint itself - why navigation waits for pointer-release, why a signature block wraps, why the tree filter ignores the search term. Comments citing an external product's documented behaviour as the source of a rule are left alone; there the reference is the reason, not a memory of how the code arrived. Assisted-by: Claude:claude-opus-5[1m]:Claude Code	1 month ago
Siegfried Pammer	76afcae7d0	Remove UI tests that only restate the code they cover These fixtures were written while porting to Avalonia, as an author's own verification step rather than as coverage: reflection asserting that a type derives from its base and that a property has the type it is declared with; literals (MinHeight 29, Padding 3, MaxWidth 900) copied out of the .axaml beside them; a property override asserted only so pane descendants stay reachable from tests. None of them can fail except when someone deliberately edits the line they mirror, and then they fail as a chore. StartupPerfTests keeps its two [Explicit] benchmarks, which print per-phase timings worth reading. The third was a wall-clock assertion (8 CoreLib copies must settle in under 15s) that ran in CI, where a shared runner decides the verdict; as [Explicit] it would be strictly dominated by the 200-assembly benchmark it was derived from, so it goes. Two fixtures are trimmed rather than deleted, because their kernel is real: XmlDocLoader's ref-pack fallback has no other test in the repo, and the MenuIcon metadata rasterisation was dropped once during the port already. Both now assert that without booting MainWindow to reach it. This is worth about two seconds - it buys reviewers less to read, not CI less to do. Assisted-by: Claude:claude-opus-5:Claude Code	1 month ago
Siegfried Pammer	51aafa0fd9	Move ILSpy UI code back to the ICSharpCode.ILSpy root namespace The Avalonia port had placed the UI app in an ILSpy.* namespace tree, while the csproj RootNamespace and every prior release (through 10.1) use ICSharpCode.ILSpy.*. Restoring the historical namespace reduces the public API diff against release/10.1 for plugin authors and removes the shadowing that forced global:: qualifiers in the test project. The Images class and AccessOverlayIcon enum move back into the root namespace (as in 10.1), since an ICSharpCode.ILSpy.Images namespace would shadow the Images class for all code inside ICSharpCode.ILSpy. Assisted-by: Claude:claude-fable-5:Claude Code	3 months ago
Siegfried Pammer	2d4722805b	Hover tooltip signature wraps instead of clipping User-reported: editor hover tooltips were not wide enough to show full method signatures. Root cause was a three-way layout interaction — DocumentationRenderer.AddSignatureBlock created the signature SelectableTextBlock with TextWrapping.NoWrap, the outer Border capped MaxWidth at 600px, and the wrapping ScrollViewer disabled horizontal scrolling. So anything past 600px was simply clipped, with no way to reveal the rest. Assisted-by: Claude:claude-opus-4-7:Claude Code	3 months ago