By moving the check after the `isLoggingEnabled()` check, there isn’t unreachable code, so the compiler warning goes away. (The `_isDebug…` check has to be second, or the `isLoggingEnabled()` call would be unreachable code.) For the other case, I wrapped the entire check in a single `assert`, avoiding the `_isDebugAssertConfiguration` call altogether. (This one wasn’t a warning, but I think this approach is more canonical, and it’s probably best to avoid `_` functions where possible.) |
||
|---|---|---|
| .. | ||
| ContactShareViewModel.swift | ||
| CVItemViewModel.h | ||
| OWSQuotedReplyModel.h | ||
| OWSQuotedReplyModel.m | ||
| ThreadUtil+SignalUI.swift | ||
| ThreadViewModel.swift | ||
| VoiceMessageModel.swift | ||