The version of BM_elem_attrs_copy that took a map used a reference the version without a map didn't, which is fairly confusing. Pass by pointer now unless this is part of a wider refactor to move to references everywhere.