@Pharylon said:The DataCleanup() method is called 18,098 times in our codebase
In terms of functionality, it certainly doesn't appear to do very much of any use, beyond heating the server room just that little bit more. Overall it's probably not much of an overhead, especially as it comes before / after database access, which should be several orders of magnitude more significant.
That said, it definitely instills a desire to pull out the cluebat and apply it to those reponsible, finger by finger, until they can't physically use a keyboard any more. The generics stuff is particularly splendid.