You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@dubbo.apache.org by "satansk (GitHub)" <gi...@apache.org> on 2019/01/31 00:58:11 UTC

[GitHub] [incubator-dubbo] satansk commented on issue #3398: move construction of ConfigChangeEvent outside the lambda expression

> Another thing, can we make the field of `ConfigChangeEvent` `final`?
> It seems that if we want to share the `ConfigChangeEvent` instance, it is a better choice to make the `ConfigChangeEvent` field `final`.

Agree, it's better to be final.

[ Full content available at: https://github.com/apache/incubator-dubbo/pull/3398 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org