Blender Git Loki

Git Commits -> Revision 1df8abf

Revision 1df8abf by Jacques Lucke (master)
November 23, 2021, 13:56 (GMT)
Geometry Nodes: add namespace for every file

This puts all static functions in geometry node files into a new
namespace. This allows using unity build which can improve
compile times significantly (P2578).

* The name space name is derived from the file name. That makes
it possible to write some tooling that checks the names later on.
The file name extension (`cc`) is added to the namespace name as
well. This also possibly simplifies tooling but also makes it more
obvious that this namespace is specific to a file.
* In the register function of every node, I added a namespace alias
`namespace file_ns = blender::nodes::node_geo_*_cc;`. This avoids
some duplication of the file name and may also simplify tooling,
because this line is easy to detect. The name `file_ns` stands for "file
namespace" and also indicates that this namespace corresponds to
the current file. In the beginning I used `node_ns` but `file_ns` is more
generic which may make it more suitable when we want to use unity
builds outside of the nodes modules in the future.
* Some node files contain code that is actually shared between
different nodes. For now I left that code in the `blender::nodes`
namespace and moved it to the top of the file (couldn't move it to
the bottom in all cases, so I just moved it to the top everywhere).
As a separate cleanup step, this shared code should actually be
moved to a separate file.

Differential Revision: https://developer.blender.org/D13330

Commit Details:

Full Hash: 1df8abff257030ba79bc23dc321f35494f4d91c5
Parent Commit: 47276b8
Lines Changed: +1348, -1041

135 Modified Paths:

/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_align_rotation_to_vector.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_clamp.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_color_ramp.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_combine_xyz.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_compare.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_convert.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_curve_map.cc (+11, -9) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_fill.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_map_range.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_math.cc (+10, -8) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_mix.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_proximity.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_randomize.cc (+42, -36) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_sample_texture.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_separate_xyz.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_transfer.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_vector_math.cc (+10, -8) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_attribute_vector_rotate.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_endpoints.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_reverse.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_select_by_handle_type.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_set_handles.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_spline_type.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_subdivide.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_curve_to_points.cc (+60, -54) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_delete_geometry.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_edge_split.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_material_assign.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_mesh_to_curve.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_points_to_volume.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_distribute.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_instance.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_rotate.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_scale.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_separate.cc (+17, -11) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_point_translate.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_raycast.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_select_by_material.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_subdivision_surface.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/legacy/node_geo_legacy_volume_to_mesh.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_attribute_capture.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_attribute_remove.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_boolean.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_bounding_box.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_collection_info.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_convex_hull.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_endpoint_selection.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_fill.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_fillet.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_handle_type_selection.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_length.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_parameter.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_bezier_segment.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_circle.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_line.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_quadratic_bezier.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_quadrilateral.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_spiral.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_star.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_resample.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_reverse.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_sample.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_set_handles.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_spline_type.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_to_mesh.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_to_points.cc (+22, -18) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_curve_trim.cc (+10, -8) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_delete_geometry.cc (+46, -40) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_distribute_points_on_faces.cc (+8, -8) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_edge_split.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_image_texture.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_curve_handles.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_curve_tilt.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_id.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_index.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_material.cc (+7, -5) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_material_index.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_normal.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_position.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_radius.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_shade_smooth.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_spline_cyclic.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_spline_length.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_spline_resolution.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_input_tangent.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_instances_to_points.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_instance_on_points.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_is_viewport.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_join_geometry.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_material_replace.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_material_selection.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_cone.cc (+81, -75) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_cube.cc (+34, -28) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_cylinder.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc (+34, -28) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_ico_sphere.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_line.cc (+45, -39) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_subdivide.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_to_curve.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_mesh_to_points.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_object_info.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_points_to_vertices.cc (+7, -5) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_points_to_volume.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_proximity.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_raycast.cc (+10, -8) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_realize_instances.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_rotate_instances.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_scale_instances.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_separate_components.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_separate_geometry.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_curve_handles.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_curve_radius.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_curve_tilt.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_id.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_material.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_material_index.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_point_radius.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_position.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_shade_smooth.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_spline_cyclic.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_string_join.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_string_to_curves.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_subdivision_surface.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_switch.cc (+9, -7) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_transfer_attribute.cc (+10, -8) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_transform.cc (+43, -37) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_translate_instances.cc (+6, -4) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_triangulate.cc (+8, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_viewer.cc (+9, -6) (Diff)
/source/blender/nodes/geometry/nodes/node_geo_volume_to_mesh.cc (+9, -7) (Diff)
Tehnyt: Miika HämäläinenViimeksi päivitetty: 07.11.2014 14:18MiikaH:n Sivut a.k.a. MiikaHweb | 2003-2021