<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 19, 2018 at 5:01 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>
I see two styles here -- get_physical_device_display_<wbr>properties* both<br>
use a helper function that takes a pointer to either of the property<br>
returns while get_physical_device_display_<wbr>plane_properties* and<br>
get_display_mode_properties* open-code things.<br>
<br>
I'm easy with which style you pick, but I think they should be the<br>
same. I have a mild preference for the second style as I think it's<br>
easier to read the code without all of the conditionals.<span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">I did that mostly because the iteration in get_physical_device_display_properties was more complicated and seemed to require "real" set-up.  It's not terrible so I'd be ok with open-coding it if needed.  Another option would be to create a temporary and call the 2 variant and then copy the elements one-by-one into the output.  Sadly, it's gross no matter which way we go.  Thoughts?</div><div class="gmail_extra"><br></div><div class="gmail_extra">--Jason<br></div></div>