> Renee Danson Sommerfeld wrote: >>> code review requested for: >>> >>> 11927 - walkprop shows properties that should be skipped >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=11927 >>> >>> http://zhadum.east/export/ws/am223141/temp/nwam1-work/webrev/ >>> >> This one makes me a little nervous, given that it's fairly complicated >> logic and recursion is now in the picture. Are we certain that there >> are clear stopping points for the recursion?
I've been thinking more about this recursion in detail. The way it currently is, the recursion relies that there are no circular dependencies in the rules table. If there is a circular dependency by mistake (I can't think of an actual example), then the recursion has a problem. I'm gonna have to include code that checks for this circularity and if detected, stops the recursion. Anurag
