Skip to content

Commit aff0ae5

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 20f9311 commit aff0ae5

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
@@ -418,13 +418,19 @@ bool _sameDay(DateTime date1, DateTime date2) {
418418
/// * When the object will no longer be used, call [dispose] to free
419419
/// resources on the [PerAccountStore].
420420
class MessageListView with ChangeNotifier, _MessageSequence {
421-
MessageListView._({required this.store, required this.narrow});
422-
423421
factory MessageListView.init(
424422
{required PerAccountStore store, required Narrow narrow}) {
425-
final view = MessageListView._(store: store, narrow: narrow);
426-
store.registerMessageList(view);
427-
return view;
423+
return MessageListView._(store: store, narrow: narrow)
424+
.._register();
425+
}
426+
427+
MessageListView._({required this.store, required this.narrow});
428+
429+
final PerAccountStore store;
430+
Narrow narrow;
431+
432+
void _register() {
433+
store.registerMessageList(this);
428434
}
429435

430436
@override
@@ -433,9 +439,6 @@ class MessageListView with ChangeNotifier, _MessageSequence {
433439
super.dispose();
434440
}
435441

436-
final PerAccountStore store;
437-
Narrow narrow;
438-
439442
/// Whether [message] should actually appear in this message list,
440443
/// given that it does belong to the narrow.
441444
///

0 commit comments

Comments
 (0)