From: Matt B. <gud...@ya...> - 2007-01-31 20:54:55
|
IMHO this is sort of big: it yields a speed improvement in the test cases, and I feel it serves a practical purpose as well. Anyway, I wanted to call attention to it so that it could be shot down if need be. I have modified BaseReflector NOT to set properties to identical or immutable-and-equals() values. This is especially important with copiers, because when you do deep copying, you often don't really need to set the value because you've only changed the state, if you will, of the same actual object instance. Where this really helps, however, is in the situation where you are copying to objects with getters but no setters. Because the actual value object is the same, Morph will now recognize that there is no need to perform the set() and thus its functionality should be extended as a result of these changes (it won't fail trying to perform sets it didn't have to do to begin with). These are revs 101 + 102. -Matt ____________________________________________________________________________________ It's here! Your new message! Get new email alerts with the free Yahoo! Toolbar. http://tools.search.yahoo.com/toolbar/features/mail/ |