Skip to content

Comments

Bug fixes in chapter 9#20

Open
dgruntz wants to merge 2 commits intoeppleton:masterfrom
dgruntz:kap-9
Open

Bug fixes in chapter 9#20
dgruntz wants to merge 2 commits intoeppleton:masterfrom
dgruntz:kap-9

Conversation

@dgruntz
Copy link
Contributor

@dgruntz dgruntz commented Aug 15, 2018

Updates the progress indicator to be finished
By invoking

    updateProgress(max, max);

the properties are informed that the task is completed.
[this should probably be added to the former projects as well.]

Fixes a bug in tweetalot (chap 9.4)
In the original version the RefreshService was declared as a local variable in the constructor and was reclaimed by the GC after creation of the timeline.

The call homeTimeline.refresh() declared in the MainApp is no longer needed (was commented out). Without that code the timeline was not initialized.

dgruntz added 2 commits August 5, 2018 22:42
By invoking
		updateProgress(max, max);
the properties are informed that the task is completed.
[this should probably be added to the former projects as well.]
In the original version the RefreshService was declared as a local
variable in the constructor and was reclaimed by the GC after creation
of the timeline.

The call homeTimeline.refresh() declared in the MainApp is no longer
needed (was commented out). Without that code the timeline was not
initialized.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant