Restriction of the nodes api to clearly define never-null function arguments. Side effects: some assertions and null-check (with early return) were removed. On the caller side is ensured to never derefer null to pass argument (mainly in RNA). In addition, one pointer argument now actually a return type. By-reference return types instead of pointers going to be separate kind of change since also imply of cleaning up variables created from reference. Also good future improvement would be to mark a copy-constructor as explicit for DNA node types. Pull Request: https://projects.blender.org/blender/blender/pulls/134627
44 lines
1.1 KiB
C++
44 lines
1.1 KiB
C++
/* SPDX-FileCopyrightText: 2006 Blender Authors
|
|
*
|
|
* SPDX-License-Identifier: GPL-2.0-or-later */
|
|
|
|
/** \file
|
|
* \ingroup nodes
|
|
*/
|
|
|
|
#include <optional>
|
|
|
|
#include "BKE_node_runtime.hh"
|
|
|
|
#include "NOD_socket_search_link.hh"
|
|
|
|
#include "node_composite_util.hh"
|
|
|
|
bool cmp_node_poll_default(const blender::bke::bNodeType * /*ntype*/,
|
|
const bNodeTree *ntree,
|
|
const char **r_disabled_hint)
|
|
{
|
|
if (!STREQ(ntree->idname, "CompositorNodeTree")) {
|
|
*r_disabled_hint = RPT_("Not a compositor node tree");
|
|
return false;
|
|
}
|
|
return true;
|
|
}
|
|
|
|
void cmp_node_update_default(bNodeTree * /*ntree*/, bNode *node)
|
|
{
|
|
node->runtime->need_exec = 1;
|
|
}
|
|
|
|
void cmp_node_type_base(blender::bke::bNodeType *ntype,
|
|
std::string idname,
|
|
const std::optional<int16_t> legacy_type)
|
|
{
|
|
blender::bke::node_type_base(*ntype, idname, legacy_type);
|
|
|
|
ntype->poll = cmp_node_poll_default;
|
|
ntype->updatefunc = cmp_node_update_default;
|
|
ntype->insert_link = node_insert_link_default;
|
|
ntype->gather_link_search_ops = blender::nodes::search_link_ops_for_basic_node;
|
|
}
|