archive/blender-archive
Archived
1
1
Fork 0
Code Pull Requests Packages Activity

Cleanup: add BLI_linklist_find_last

Browse Source 
This makes adding to the end of a linked list simpler,
In most cases we avoid this in favor of BLI_linklist_append.

For one off operations it's OK.
This commit is contained in:
Campbell Barton
2020-07-10 12:04:29 +10:00
parent 3dd460aa7f
commit 8f24ec2e26
3 changed files with 15 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
source/blender/editors/uvedit/uvedit_path.c
View File

@@ -265,10 +265,10 @@ static void mouse_mesh_uv_shortest_path_vert(Scene *scene,
if (path) {
if ((l_dst_add_to_path != NULL) && (BLI_linklist_index(path, l_dst_add_to_path) == -1)) {
/* Weak, we could find the last and append after that. */
BLI_linklist_reverse(&path);
BLI_linklist_prepend(&path, l_dst_add_to_path);
BLI_linklist_reverse(&path);
/* Append, this isn't optimal compared to #BLI_linklist_append, it's a one-off lookup. */
LinkNode *path_last = BLI_linklist_find_last(path);
BLI_linklist_insert_after(&path_last, l_dst_add_to_path);
BLI_assert(BLI_linklist_find_last(path)->link == l_dst_add_to_path);
}
/* toggle the flag */