Skip to content

Commit

Permalink
[bgen] Fix compiler warnings. (#21184)
Browse files Browse the repository at this point in the history
And make warnings report as errors, so this doesn't happen again.
  • Loading branch information
rolfbjarne committed Sep 9, 2024
1 parent 52ee455 commit 7325ce8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
2 changes: 1 addition & 1 deletion tests/bgen/bgen-tests.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<RootNamespace>bgen_tests</RootNamespace>

<IsPackable>false</IsPackable>
<WarningsAsErrors>Nullable</WarningsAsErrors>
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
</PropertyGroup>

<ItemGroup>
Expand Down
10 changes: 1 addition & 9 deletions tests/generator/BGenTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1701,15 +1701,7 @@ public void BackingFieldType (Profile profile)
public void UnderlyingFieldType (Profile profile)
{
Configuration.IgnoreIfIgnoredPlatform (profile.AsPlatform ());
var bgen = BuildFile (profile, true, true, "tests/underlyingfieldtype.cs");

#if NET
const string nintName = "System.IntPtr";
const string nuintName = "System.UIntPtr";
#else
const string nintName = "System.nint";
const string nuintName = "System.nuint";
#endif
BuildFile (profile, true, true, "tests/underlyingfieldtype.cs");
}

[Test]
Expand Down

4 comments on commit 7325ce8

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

Please sign in to comment.