<div dir="ltr">Thank you for the feedback.  I have a practical question - what is the workflow for making changes to this module, now that it's been merged?  Do I clone, then branch, edit, commit, push, merge, and finally mail a patch to the list?  I'm still learning git.  I've been on the rcs/cvs/svn bandwagon for a long time.<br><br>I'd like some clarification on the need for portability.  Once I read your comments, I came up with an ad-hoc list of platforms I felt I could reasonably support: major Linux distros, FreeBSD, OS X, and OmniOS.  I tried the module on each successfully, though ccanlint needed changes to build on everything but Linux, and the failtest based test went rogue on two of the platforms.<div><br></div><div>Compound literals are a feature of C99 (nice article: <a href="http://www.drdobbs.com/the-new-c-compound-literals/184401404">http://www.drdobbs.com/the-new-c-compound-literals/184401404</a>).  Intermingled declarations and code are also from C99.  Half of the platforms above use gcc and half use clang.  Both gcc and clang support statement expressions.  The _XOPEN_SOURCE define in _info is there for getline(3) on Linux which is used solely in the demo code in _info.</div><div><br></div><div>Is there a set of platforms CCAN targets for support?  Is there some way I should document that the code is C99?  Would the lines below be the right way to handle the statement expression dependance?</div><br>#if !HAVE_STATEMENT_EXPR <br>#error Sorry, deque requires statement expressions <br>#endif<div><br></div><div>Thanks.  -Dan</div><div><br><div><div class="gmail_quote"><div dir="ltr">On Sun, Dec 27, 2015 at 10:45 PM David Gibson <<a href="mailto:david@gibson.dropbear.id.au">david@gibson.dropbear.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Dec 23, 2015 at 12:11:44PM +0000, Dan Good wrote:<br>
> deque - type-preserving resizing circular deque<br>
<br>
Concept looks good.  There are some fairly minor nits in the<br>
implementation that I've noted below.  However, none are serious<br>
enough to hold up merging, so I've applied this.  I did modify<br>
trivially to remove some trailing whitespace.<br>
<br>
It would be nice to address some of the comments in follow up<br>
commits, particularly some of the portability issues.<br></blockquote><div> [...]</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--<br>
David Gibson                    | I'll have my music baroque, and my code<br>
david AT <a href="http://gibson.dropbear.id.au" rel="noreferrer" target="_blank">gibson.dropbear.id.au</a>  | minimalist, thank you.  NOT _the_ _other_<br>
                                | _way_ _around_!<br>
<a href="http://www.ozlabs.org/~dgibson" rel="noreferrer" target="_blank">http://www.ozlabs.org/~dgibson</a><br>
</blockquote></div></div></div></div>