Rename bctbx_list_remove_link() into bctbx_list_unlink() and bctbx_list_delete_link() into bctbx_list_erase_link().
Indeed, there was a confusion between these two methods when the MSList API was deprecated, resulting in thousands of small memory leaks.
bctbx_list_remove_link() is not freeing the list node, unlike bctbx_list_delete_link.