<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 19, 2018 at 5:09 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"><span class="">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> Instead, we can just use get_capabilities2.  This way back-ends only<br>
> have to implement one hook.<br>
<br>
</span>Yeah, this looks nice. Are you going to remove the unused functions at<br>
some point?<br></blockquote><div><br></div><div>They're being used as helpers and they're actually kind-of nice for that.  Other places where we've implemented chained versions of things it's really nice to have one helper for each type of link in the chain.  Those just become for the first link.  We could inline them if needed but I haven't seen a point.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Keith Packard <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
-keith<br>
</font></span></blockquote></div><br></div></div>