Skip to content

Commit 7b7021e

Browse files
committed
msglist [nfc]: Rearrange to follow normal ordering of class members
In particular this causes the handful of places where each field of MessageListView needs to appear to all be next to each other.
1 parent fd18923 commit 7b7021e

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

lib/model/message_list.dart

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -448,13 +448,19 @@ bool _sameDay(DateTime date1, DateTime date2) {
448448
/// * When the object will no longer be used, call [dispose] to free
449449
/// resources on the [PerAccountStore].
450450
class MessageListView with ChangeNotifier, _MessageSequence {
451-
MessageListView._({required this.store, required this.narrow});
452-
453451
factory MessageListView.init(
454452
{required PerAccountStore store, required Narrow narrow}) {
455-
final view = MessageListView._(store: store, narrow: narrow);
456-
store.registerMessageList(view);
457-
return view;
453+
return MessageListView._(store: store, narrow: narrow)
454+
.._register();
455+
}
456+
457+
MessageListView._({required this.store, required this.narrow});
458+
459+
final PerAccountStore store;
460+
Narrow narrow;
461+
462+
void _register() {
463+
store.registerMessageList(this);
458464
}
459465

460466
@override
@@ -463,9 +469,6 @@ class MessageListView with ChangeNotifier, _MessageSequence {
463469
super.dispose();
464470
}
465471

466-
final PerAccountStore store;
467-
Narrow narrow;
468-
469472
/// Whether [message] should actually appear in this message list,
470473
/// given that it does belong to the narrow.
471474
///

0 commit comments

Comments
 (0)