<div dir="ltr"><div><div><div><div><div>Hi Steve,<br><br></div>Another smallpoint to resolve is around the logic for the support of the second declaration in SIP files, e.g. the bit in [] brackets here:<br><br>   KFontChooser(QWidget* parent /TransferThis/ = 0, const KFontChooser::DisplayFlags& flags = KFontChooser::DisplayFrame, const QStringList& fontList = QStringList(), int visibleListSize = 8, Qt::CheckState* sizeIsRelativeState = 0)<br>        [(QWidget* = 0, const KFontChooser::DisplayFlags& = KFontChooser::DisplayFrame, const QStringList& = QStringList(), int = 8, Qt::CheckState* = 0)];<br><br></div>Now, until yesterday the rule for defining the parameters for the main and the second declarations were both strings, but as I rebased, I needed to support the change you made where the former is now an array.<br><br></div>I suggest that for consistency the second set of parameters should also be an array:<br><br>     "getFontAndText":<br>     {<br>-        "parameters": "QFont& theFont, QString& theString, const KFontChooser::DisplayFlags& flags = KFontChooser::NoDisplayFlags, QWidget* parent /Transfer/ = 0, Qt::CheckState sizeIsRelativeState = Qt::Unchecked",<br>+        "parameters": ["QFont& theFont", "QString& theString", "const KFontChooser::DisplayFlags& flags = KFontChooser::NoDisplayFlags", "QWidget* parent /Transfer/ = 0", "Qt::CheckState sizeIsRelativeState = Qt::Unchecked"],<br>         "fn_result": "SIP_PYTUPLE",<br>-        "decl2": "QFont&, QString&, const KFontChooser::DisplayFlags& = KFontChooser::NoDisplayFlags, QWidget* = 0, Qt::CheckState* = 0",<br>+        "decl2": ["QFont&", "QString&", "const KFontChooser::DisplayFlags& = KFontChooser::NoDisplayFlags", "QWidget* = 0", "Qt::CheckState* = 0"],<br>         "fn_result2": "int",<br>         "code":<br>         """<br><br></div>I've coded this up but wanted to check for the impact this might have on you. Please let me know if this is going to be a problem.<br><br></div>Thanks, Shaheed<br><div><br><br><div><div><br><br></div></div></div></div>