Re: [PATCH v2] score: Replace priority prepend it with an enum

2021-08-12 Thread Chris Johns
+1 On 13/8/21 3:43 am, Gedare Bloom wrote: > looks good to me. > > On Thu, Aug 12, 2021 at 8:26 AM Sebastian Huber > wrote: >> >> Use the new Priority_Group_order enum instead of a boolean to indicated if a >> priority should be inserted as the first or last node into its priority >> group. >>

Re: [PATCH v2] score: Replace priority prepend it with an enum

2021-08-12 Thread Gedare Bloom
looks good to me. On Thu, Aug 12, 2021 at 8:26 AM Sebastian Huber wrote: > > Use the new Priority_Group_order enum instead of a boolean to indicated if a > priority should be inserted as the first or last node into its priority group. > This makes the code more expressive. It is also a bit more

[PATCH v2] score: Replace priority prepend it with an enum

2021-08-12 Thread Sebastian Huber
Use the new Priority_Group_order enum instead of a boolean to indicated if a priority should be inserted as the first or last node into its priority group. This makes the code more expressive. It is also a bit more efficient since a branch in _Scheduler_Node_set_priority() is avoided and a simple