Code review comment for lp:~widelands-dev/widelands/ai-scheduler

When using simple std::pair<ScheduleTasks, DueTime>, I would completely eliminate the scheduler_review() function. It would made the code shorter and a bunch of variables would disappear.