Hans Goudey
f1c0249f34
This patch moves material indices from the mesh `MPoly` struct to a generic integer attribute. The builtin material index was already exposed in geometry nodes, but this makes it a "proper" attribute accessible with Python and visible in the "Attributes" panel. The goals of the refactor are code simplification and memory and performance improvements, mainly because the attribute doesn't have to be stored and processed if there are no materials. However, until 4.0, material indices will still be read and written in the old format, meaning there may be a temporary increase in memory usage. Further notes: * Completely removing the `MPoly.mat_nr` after 4.0 may require changes to DNA or introducing a new `MPoly` type. * Geometry nodes regression tests didn't look at material indices, so the change reveals a bug in the realize instances node that I fixed. * Access to material indices from the RNA `MeshPolygon` type is slower with this patch. The `material_index` attribute can be used instead. * Cycles is changed to read from the attribute instead. * BMesh isn't changed in this patch. Theoretically it could be though, to save 2 bytes per face when less than two materials are used. * Eventually we could use a 16 bit integer attribute type instead. Ref T95967 Differential Revision: https://developer.blender.org/D15675
94 lines
3.0 KiB
C++
94 lines
3.0 KiB
C++
/* SPDX-License-Identifier: GPL-2.0-or-later
|
|
* Copyright 2001-2002 NaN Holding BV. All rights reserved. */
|
|
|
|
#pragma once
|
|
|
|
/** \file
|
|
* \ingroup bke
|
|
*/
|
|
|
|
#include "BLI_utildefines.h"
|
|
|
|
#ifdef __cplusplus
|
|
extern "C" {
|
|
#endif
|
|
|
|
struct CustomData;
|
|
struct Mesh;
|
|
struct MFace;
|
|
|
|
/**
|
|
* Convert the hidden element attributes to the old flag format for writing.
|
|
*/
|
|
void BKE_mesh_legacy_convert_hide_layers_to_flags(struct Mesh *mesh);
|
|
/**
|
|
* Convert the old hide flags (#ME_HIDE) to the hidden element attribute for reading.
|
|
* Only add the attributes when there are any elements in each domain hidden.
|
|
*/
|
|
void BKE_mesh_legacy_convert_flags_to_hide_layers(struct Mesh *mesh);
|
|
|
|
/**
|
|
* Move material indices from a generic attribute to #MPoly.
|
|
*/
|
|
void BKE_mesh_legacy_convert_material_indices_to_mpoly(struct Mesh *mesh);
|
|
/**
|
|
* Move material indices from the #MPoly struct to a generic attributes.
|
|
* Only add the attribute when the indices are not all zero.
|
|
*/
|
|
void BKE_mesh_legacy_convert_mpoly_to_material_indices(struct Mesh *mesh);
|
|
|
|
/**
|
|
* Recreate #MFace Tessellation.
|
|
*
|
|
* \note This doesn't use multi-threading like #BKE_mesh_recalc_looptri since
|
|
* it's not used in many places and #MFace should be phased out.
|
|
*/
|
|
|
|
void BKE_mesh_tessface_calc(struct Mesh *mesh);
|
|
|
|
void BKE_mesh_tessface_ensure(struct Mesh *mesh);
|
|
|
|
void BKE_mesh_add_mface_layers(struct CustomData *fdata, struct CustomData *ldata, int total);
|
|
|
|
/**
|
|
* Rotates the vertices of a face in case v[2] or v[3] (vertex index) is = 0.
|
|
* this is necessary to make the if #MFace.v4 check for quads work.
|
|
*/
|
|
int BKE_mesh_mface_index_validate(struct MFace *mface,
|
|
struct CustomData *mfdata,
|
|
int mfindex,
|
|
int nr);
|
|
|
|
void BKE_mesh_convert_mfaces_to_mpolys(struct Mesh *mesh);
|
|
|
|
/**
|
|
* The same as #BKE_mesh_convert_mfaces_to_mpolys
|
|
* but oriented to be used in #do_versions from `readfile.c`
|
|
* the difference is how active/render/clone/stencil indices are handled here.
|
|
*
|
|
* normally they're being set from `pdata` which totally makes sense for meshes which are already
|
|
* converted to #BMesh structures, but when loading older files indices shall be updated in other
|
|
* way around, so newly added `pdata` and `ldata` would have this indices set
|
|
* based on `fdata` layer.
|
|
*
|
|
* this is normally only needed when reading older files,
|
|
* in all other cases #BKE_mesh_convert_mfaces_to_mpolys shall be always used.
|
|
*/
|
|
void BKE_mesh_do_versions_convert_mfaces_to_mpolys(struct Mesh *mesh);
|
|
|
|
/* Inlines */
|
|
|
|
/* NOTE(@sybren): Instead of -1 that function uses ORIGINDEX_NONE as defined in BKE_customdata.h,
|
|
* but I don't want to force every user of BKE_mesh.h to also include that file. */
|
|
BLI_INLINE int BKE_mesh_origindex_mface_mpoly(const int *index_mf_to_mpoly,
|
|
const int *index_mp_to_orig,
|
|
const int i)
|
|
{
|
|
const int j = index_mf_to_mpoly[i];
|
|
return (j != -1) ? (index_mp_to_orig ? index_mp_to_orig[j] : j) : -1;
|
|
}
|
|
|
|
#ifdef __cplusplus
|
|
}
|
|
#endif
|