<div class="gmail_quote">On Fri, Feb 24, 2012 at 1:27 PM, Campbell Barton <span dir="ltr"><<a href="mailto:ideasman42@gmail.com" target="_blank">ideasman42@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

you have a point - but the way I see it, its implicit with many python<br>
api's that - attributes provide direct access and functions have<br>
side-effects. </blockquote><div><br>Yep, unless the attribute is actually a property, but then that's usually expected (e.g. ORM wrappers that performs operations on the underlying data storage).<br> <br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

in this case the side effect of calling select is to<br>
update selection of other geometry.<br>
So while its not great, its not totally un-pythonic IMHO.<br></blockquote><div><br>Sure, and let me make clear that I'm not complaining, the way it is now is already good. It is just that I see room for a little improvement.<br>

<br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

select_set(select, flush=False) is more clear and can work, but is<br>
redundant since you may as well just assign .select = ... in that<br>
case.<br></blockquote><br></div>I agree. The problem is that if the 'flush' argument is removed then 'set' isn't simply 'set' anymore.<br>Some suggestions:<br>.propagate_select(select)<br>.flush_select(select)<br>

.set_linked_select(select)  # not good, probably misleading?<br>.set_select(select)  # Verb-Object, consistent with other methods such as calc_face_angle<br><br>Cheers,<br><br>Daniel<br>