<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 19, 2018 at 10:05 PM, Keith Packard <span dir="ltr"><<a href="mailto:keithp@keithp.com" target="_blank">keithp@keithp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> Thoughts?<br>
<br>
You've looked at the code more closely than I have; please feel free to<br>
leave it if you think it would seem worse as separate functions.<span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">I just sent a v2 which allocates a temporary array, calls properties2, and then copies it back over.  It doesn't duplicate the iteration code and instead just leverages propertie2.  On the down side, it's a bit more allocation and data motion but, compared to the ioctl that properties2 is doing, it shouldn't be noticeable.  Let me know what you think of that version.</div><div class="gmail_extra"><br></div><div class="gmail_extra">--Jason<br></div></div>