refactor: use ParYield in RedBlackTree

[jaschdoc]

Should wait for #4913
Closes #4867

[magnus-madsen]

@jaschdoc Can I merge this even though something is broken or?

[jaschdoc]

Are you referring to #4917 ?

[jaschdoc]

@jaschdoc Can I merge this even though something is broken or?

Nothing should be broken on this branch :)

[magnus-madsen]

• Conflicts (with Matt's apologies; bad timing).
• left -> l, right -> r
• this -> v or nv or something. Just not this or val or some Scala keyword :D

[jaschdoc]

Any thoughts on indentation of yield?

[mlutze]

Any thoughts on indentation of yield?

Here's the way I would do it:

par (
    x <- e1;
    y <- e2;
) yield (x, y)

Which matches Scala style

[magnus-madsen]

Any thoughts on indentation of yield?

Here's the way I would do it:

par (
    x <- e1;
    y <- e2;
) yield (x, y)

Which matches Scala style

Yes, but if there are few elements (like two) its OK to keep it on the same line IMO

[jaschdoc]

Alright done

[magnus-madsen]

Pretty nice cleanup, right? 👍

[jaschdoc]

Yeah very