<div dir="ltr"><div><div><div>Just to close this thread...in the review I just posted, I renamed the stuff like this:<br></div><br></div>decl2 -> cxx_decl<br></div>fn_result2 -> cxx_fn_result<br><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 21 January 2017 at 12:34, Shaheed Haque <span dir="ltr"><<a href="mailto:srhaque@theiet.org" target="_blank">srhaque@theiet.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Thanks for confirming.<br><br></div>As for the second declaration etc, the SIP specification at <a href="http://pyqt.sourceforge.net/Docs/sip4/specification_files.html" target="_blank">http://pyqt.sourceforge.net/<wbr>Docs/sip4/specification_files.<wbr>html</a> refers to it as the "<em>c++-signature</em>", so I'll look into replacing the number with a suffix like "_cpp".<br><br></div>Thanks, Shaheed<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 21 January 2017 at 10:58, Stephen Kelly <span dir="ltr"><<a href="mailto:steveire@gmail.com" target="_blank">steveire@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>Shaheed Haque wrote:<br>
<br>
> Hi Steve,<br>
><br>
> Another smallpoint to resolve is around the logic for the support of the<br>
> second declaration in SIP files, e.g. the bit in [] brackets here:<br>
<br>
</span><span>> I suggest that for consistency the second set of parameters should also be<br>
> an array:<br>
<br>
</span>That makes sense to me.<br>
<br>
I'm not a fan of numbers in interfaces though. Can you choose alternate<br>
names which convey what the second set of values are for?<br>
<br>
Thanks,<br>
<br>
Steve.<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>