search

reactive-streams / reactive-streams-dotnet

Reactive Streams for .NET
MIT No Attribution
197 stars 28 forks source link

Replaced NullReferenceException with ArgumentNullException #23

Closed marcpiechura closed 7 years ago

marcpiechura commented 8 years ago

We used ArgumentNullException in these cases because it describes the cause more precisely than a NullReferenceException and has build in support for the parameter name.

akarnokd commented 8 years ago

I'd accept both as a valid response to a null in any tests. Usually we don't bother with null tests in cases when a dereference would throw NPE immediately.