Changeset View
Changeset View
Standalone View
Standalone View
sys/compat/linuxkpi/common/src/linux_radix.c
Context not available. | |||||
} | } | ||||
} while (height != -1); | } while (height != -1); | ||||
iter->index = index; | iter->index = index; | ||||
iter->next_index = index++; | |||||
hselasky: this looks like a bug? ++index ?? See comments below. | |||||
Done Inline ActionsYes. I simplified the code as you suggested. johalun0_gmail.com: Yes. I simplified the code as you suggested.
| |||||
return (true); | return (true); | ||||
} | } | ||||
Context not available. | |||||
return (item); | return (item); | ||||
} | } | ||||
void | |||||
radix_tree_iter_delete(struct radix_tree_root *root, | |||||
struct radix_tree_iter *iter, void **slot) | |||||
{ | |||||
if (radix_tree_delete(root, iter->index)) { | |||||
Done Inline ActionsThis increment is done by the caller and should be removed! hselasky: This increment is done by the caller and should be removed! | |||||
iter->index = iter->next_index; | |||||
Done Inline Actionsiter->index++ ?? hselasky: iter->index++ ?? | |||||
} | |||||
} | |||||
int | int | ||||
radix_tree_insert(struct radix_tree_root *root, unsigned long index, void *item) | radix_tree_insert(struct radix_tree_root *root, unsigned long index, void *item) | ||||
{ | { | ||||
Context not available. |
this looks like a bug? ++index ?? See comments below.