DirtyCardQueueSet has a "helper" function apply_closure_to_completed_buffer_helper. It is only called by apply_closure_to_completed_buffer, and yet is public. And spliting up the functionality into the outer caller and the internal helper actually complicates and obfuscates what is going on. It would be better if the functionality of the helper were merged into its caller and the helper eliminated as a separate function.