[domire8]

Description

With #173, we introduced a helper is_node_initialized in the controller interface. However, it was not consistently used and also doesn't bring any benefit since it was just catching the runtime error from here and rethrowing with a different error. If anything, we would have to do that on each get_node() (and there are many of those) so I'm in favor of not doing that at all. Anyways, such problems would only arise if someone was do to something weird in a constructor of a controller.

I also ran a few tests, we never run into segfaults with get_node() which I was initially afraid of.

Review guidelines

Estimated Time of Review: 5 minutes

Checklist before merging:

• Confirm that the relevant changelog(s) are up-to-date in case of any user-facing changes

Related issues

Blocks:

• Add TF listener and broadcaster in controller base class #244

[eeberhard]

this will need a breaking changelog entry before going into main

edit: I see it was only introduced in the base branch anyway, never mind