which of these would you prefer: ``` when (targetHandler) { is HandlerCollection -> targetHa...

# codereview

[tipsy]

which of these would you prefer:

when (targetHandler) {
    is HandlerCollection -> targetHandler.apply { addHandler(defaultHandlers) }
    is HandlerWrapper -> targetHandler.apply { handler = defaultHandlers}
}

and

(targetHandler as? HandlerCollection)?.apply { addHandler(defaultHandlers) }
(targetHandler as? HandlerWrapper)?.apply { handler = defaultHandlers }

[Spike Baylor]

For my own knowledge, on the when block do you actually need to use apply in this case? or could you just call targetHandler.addHandler(defaultHandlers) and targetHandler.handler = defaultHandlers

[tipsy]

thanks, will fix that