Skip to content

Commit 028e2a7

Browse files
committed
narrow [nfc]: Use a switch statement in MentionsNarrow.containsMessage
This makes more visually clear the division between the list of flags that count, and the list of flags that don't.
1 parent 8c07c48 commit 028e2a7

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

lib/model/narrow.dart

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -291,17 +291,21 @@ class MentionsNarrow extends Narrow {
291291

292292
@override
293293
bool containsMessage(Message message) {
294-
return message.flags.any((flag) =>
294+
return message.flags.any((flag) {
295295
switch (flag) {
296-
MessageFlag.mentioned => true,
297-
MessageFlag.wildcardMentioned => true,
298-
MessageFlag.read => false,
299-
MessageFlag.starred => false,
300-
MessageFlag.collapsed => false,
301-
MessageFlag.hasAlertWord => false,
302-
MessageFlag.historical => false,
303-
MessageFlag.unknown => false,
304-
});
296+
case MessageFlag.mentioned:
297+
case MessageFlag.wildcardMentioned:
298+
return true;
299+
300+
case MessageFlag.read:
301+
case MessageFlag.starred:
302+
case MessageFlag.collapsed:
303+
case MessageFlag.hasAlertWord:
304+
case MessageFlag.historical:
305+
case MessageFlag.unknown:
306+
return false;
307+
}
308+
});
305309
}
306310

307311
@override

0 commit comments

Comments
 (0)