I don't see it in yours unless you're referencing the nice one you wrote on XModuleDescriptor which may suffice now. I believe the reason for this one was to make the table_of_contents comparison less fragile (as to whether it succeeds w/o breaking eq semantics, I'm not sure.)