Why is this a TODO? Is this not already the "Dispose pattern"?