<div dir="ltr">Somewhere we have to have the logic as to which users we can handle (those which are embedded in a module). We can have that in the callback with a boolean flag as to whether we could handle it, or we could have that logic in the foreach function. The latter seemed cleaner to me, but as I said, no strong opinions here.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 23, 2015 at 7:12 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><p dir="ltr"></p>
<p dir="ltr">> > How about having a single callback that takes a Value and do the cast in the callback?<br>
> My concern was that I already have to check for the type here in order to see whether to call the callback or recurse, so I thought it was cleaner to have two callbacks, but I don't feel very strongly about that.</p>
</span><p dir="ltr">Why not always call the callback? We have a set to avoid a loop.</p>
<p dir="ltr">Cheers, Rafael<br>
</p>
</blockquote></div><br></div>