[nfc] Squash release build warnings

[airspeedswift]

In a couple of these I'm skipping creating the temporary variable and just putting the call in the assert, dunno if that's accepted style, seems cleaner than having to add another line.

[airspeedswift]

@swift-ci please smoke test

[jrose-apple]

Putting the call in the assert is fine. I suspect the people who wrote these assertions tried to keep the entire thing on one line, but we have multi-line asserts elsewhere. What's not standard is your line wrapping. :-) We're not completely consistent about whether the && goes at the end of the previous line or the start of the subsequent line, but in the latter case the indentation aligns the && with the first line, not the right-side argument.

[airspeedswift]

@swift-ci please test

[swift-ci]

Build failed
Swift Test OS X Platform
Git Sha - 2ea14cdf196e4f9dcb0734362da6a8c8ab404697

[swift-ci]

Build failed
Swift Test Linux Platform
Git Sha - 2ea14cdf196e4f9dcb0734362da6a8c8ab404697

[airspeedswift]

@swift-ci please test

[swift-ci]

Build failed
Swift Test OS X Platform
Git Sha - ad1c9d29856493cfaba68411a83d7efd4dcb55b0

[swift-ci]

Build failed
Swift Test Linux Platform
Git Sha - ad1c9d29856493cfaba68411a83d7efd4dcb55b0

[airspeedswift]

@jrose-apple this better?

[jrose-apple]

Yep, thanks.