From 85fde25178b552db4f98ae5317a825b230e13a2f Mon Sep 17 00:00:00 2001 From: Yiming Wu Date: Mon, 13 Jun 2022 21:08:36 +0800 Subject: LineArt: General code cleanups. - Use uintxx_t for all 8/16/64 bit integer types. - Removed prepend_edge_direct thingy which is no longer needed in current edge iterator model. - Minor code path adjustments like only copies view vector when necessary etc. - Correctly handle ies==NULL in edge cutting function. - White spaces and comments etc. Reviewed By: Sebastian Parborg (zeddb) Differential Revision: https://developer.blender.org/D15181 --- source/blender/makesdna/DNA_lineart_types.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'source/blender/makesdna') diff --git a/source/blender/makesdna/DNA_lineart_types.h b/source/blender/makesdna/DNA_lineart_types.h index df496269644..1cdb021cca2 100644 --- a/source/blender/makesdna/DNA_lineart_types.h +++ b/source/blender/makesdna/DNA_lineart_types.h @@ -48,9 +48,9 @@ typedef enum eLineartEdgeFlag { LRT_EDGE_FLAG_INTERSECTION = (1 << 4), LRT_EDGE_FLAG_LOOSE = (1 << 5), /* LRT_EDGE_FLAG_FOR_FUTURE = (1 << 7), */ - /* Limited to 8 bits for edge type flag, don't add anymore because `BMEdge->head.eflag` only has - * 8 bits. So unless we changed this into a non-single-bit flag thing, we keep it this way. */ - /** Also used as discarded line mark. */ + /* It's a legacy limit of 8 bits for feature lines that come from original mesh edges. It should + not be needed in current object loading scheme, but might still be relevant if we are to + impelment EditMesh loading, so don't exceed 8 bits just yet. */ LRT_EDGE_FLAG_CHAIN_PICKED = (1 << 8), LRT_EDGE_FLAG_CLIPPED = (1 << 9), /** Limited to 16 bits for the entire thing. */ -- cgit v1.2.3