Jake Wharton
Jake Wharton
> The reason `isNotNull()` exists is because otherwise each assertion would have to make a decision on how to handle nulls. Assertions that operate on actual values are defined on...
Yes. That is a fantastically succinct framing of it.
You can shard the content into additional functions after so many elements. That way only these extreme cases pay for it.
De-duplicating the exceptions to the allow-list would be a good first step toward this.
Wow we have a lot of these...
Finding some good IntelliJ bugs/feature requests at least!
Retrofit has no threading so it is not possible to do both. Factory overloads to switch to streamed bodies instead of buffered bodies is really the only option.
Depending on what execution mechanism you were using, it _may_ be possible to apply threading earlier through a custom `CallAdapter` in which you brought you own thread, updated the streaming...
Related: #1442
The place to fix this, if it indeed is worth changing, would be in the framework, not some arbitrary subclass. But keep in mind that the nullability of these methods...