Skip to content

Commit

Permalink
[bgen] Add warning to the list of exceptions instead of showing it ri…
Browse files Browse the repository at this point in the history
…ght away. (#21027)

This way execution won't stop if the warning was turned into an error, and
we'll get a list of all the warnings by executing once, instead of having to
re-execute after fixing each warning.
  • Loading branch information
rolfbjarne committed Sep 4, 2024
1 parent 87f28d5 commit d77a4dd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/bgen/Generator.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4563,8 +4563,8 @@ void GenerateMethod (MemberInformation minfo)
argCount++;
}
if (minfo.Method.GetParameters ().Length != argCount) {
ErrorHelper.Warning (1105,
minfo.selector, argCount, minfo.Method, minfo.Method.GetParameters ().Length);
exceptions.Add (ErrorHelper.CreateWarning (1105,
minfo.selector, argCount, minfo.Method, minfo.Method.GetParameters ().Length));
}
}

Expand Down

3 comments on commit d77a4dd

@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.