Public API Analyzer considers IntPtrs in CS as nints resulting in RS0017 #6059
Comments
jaredpar
pushed a commit
to jaredpar/roslyn
that referenced
this issue
Apr 11, 2023
Work around a bug in the public API analyzer dotnet/roslyn-analyzers#6059
jaredpar
pushed a commit
to dotnet/roslyn
that referenced
this issue
Apr 13, 2023
* Source build fixes The source build produced from our official build needs to include both the net7.0 and net8.0 packages. Those packages feed into the soucre build legs of other repos and they can be targeting either net7.0 or net8.0. * fixes * Fix Work around a bug in the public API analyzer dotnet/roslyn-analyzers#6059 * nint * fix package * fix
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Analyzer
Diagnostic ID: RS0017:
Symbol is part of the declared API, but is either not public or could not be foundAnalyzer source
SDK: Built-in CA analyzers in .NET 7 SDK or later
Version: SDK 7.0.100-preview.5.22307.18 - the latest Arcade
Describe the bug
CI build failure in winforms repo: https://github.com/dotnet/winforms/pull/7376/checks?check_run_id=7263703283
src\System.Windows.Forms.Primitives\src\System\Windows\Forms\Message.cs#L31
src\System.Windows.Forms.Primitives\src\System\Windows\Forms\Message.cs(31,30): error RS0016: (NETCORE_ENGINEERING_TELEMETRY=Build) Symbol 'HWnd.get' is not part of the declared API.
These errors are generated for code like this:
public IntPtr HWnd { get; set; }
because the current API is declared in PublicApi.Shipped.txt like this:
System.Windows.Forms.Message.HWnd.get -> IntPtr
but the analyzer expects:
System.Windows.Forms.Message.HWnd.get -> nint
Potentially related to: dotnet/csharplang#6065
Steps To Reproduce
See build failures in the dependency flow PR
Expected behavior
Public APIs that use IntPtr types should not be flagged as changed due to compiler change
Actual behavior
Build break
Additional context
The text was updated successfully, but these errors were encountered: