OutlinePass setSelection performs clear operation in incorrect order

[stevesch]

I believe for the intent of OutlinePass.setSelection,

this.clearSelection() in OutlinePass.js should be performed before the "for" loop that sets the newly selected items' layer.
(current line 373, move to line 364, just after the two "const" declarations)

[vanruesc]

Hi,

thanks for taking the time to point this out!

You're right, the current implementation incorrectly assumes that the given list of objects doesn't contain objects that are already selected. As a result, the selection layer will unintentionally be disabled for objects that are in both lists.

I'll fix this ASAP.

[vanruesc]

Fixed in postprocessing@4.4.1.