Another thought: Having evaluate_definitions both do the action and return a boolean indicating if anything needed evaluation is clunky. Maybe it should take the list of resources to be evaluate as an argument (which would come from unevaluated_resources) and moving the "are there any?" test up to evaluate_generators, which should simplify it as well (ultimately even making it a while loop instead of the loop with a break & done boolean).
But that's out of scope for this ticket. -- M ----------------------------------------------------------- The power of accurate observation is commonly called cynicism by those who have not got it. ~George Bernard Shaw ------------------------------------------------------------ -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
