Avoid not_done naming conflict

In order to make understanding the
usage of the 'not_done' variable easier
allow the schedule to take in the list
it will be appending to instead of relying
on a function local variable (which is
overwritten in the contained while loop)

Change-Id: I60c0c970d8fb444983e0d8b567ebe82c47e13d96
Merge "Avoid not_done naming conflict"