[Kde-bindings] smokegen: Qt 5 support added

[Michael Lawrence]

Hi Dimitar,

Has there been any more progress in updating smokegen for qt5? I made
pretty much the same changes as you to get things to compile (before I knew
about the branch), but it was segfaulting for me on qtcore, until I made
this very likely damaging hack:

diff --git a/kdebindings/generator/type_compiler.cpp
b/kdebindings/generator/type_compiler.cpp
index 6ebf602..556b1d3 100644
--- a/kdebindings/generator/type_compiler.cpp
+++ b/kdebindings/generator/type_compiler.cpp
@@ -191,6 +191,9 @@ void
TypeCompiler::visitParameterDeclaration(ParameterDeclarationAST* node)

 void TypeCompiler::visitPtrOperator(PtrOperatorAST* node)
 {
+    if (node->op == 0) {
+        return;
+    }
     if (token_text(m_session->token_stream->kind(node->op))[0] == '*') {
         QPair<bool, bool> cv = m_visitor->parseCv(node->cv);
         pointerDepth.append(cv.first);

Although it completed, the generated files contain only bindings for
QGlobalSpace. I realize that I'm not giving enough details, but I'm just
wondering if you've had more luck.

Thanks,
Michael


On Sat, Jan 26, 2013 at 11:55 AM, Dimitar Dobrev <dpldobrev at yahoo.com>wrote:

>
>     Hi all,
>
>     I've just pushed a new branch called Qt5.0.1 to smokegen. It enables
> smokegen to compile against Qt 5.0.1. For now it's been tested only on
> Windows using this installer<http://releases.qt-project.org/digia/5.0.1/latest/qt-windows-opensource-5.0.1-mingw47_32-x86-offline-2013-01-25-421.exe>
> .
>     Note: CMake failed to find QtCore to include in
> parser/kdevvarlengtharray.h so I threw out KDevVarLengthArray and
> replaced it with QVector. As far as I've seen, KDevVarLengthArray only
> has some additional template type for capacity so I guess it's a bit more
> optimal or something. Anyway, I'm not wasting another second with CMake. If
> somebody would like to revert to KDevVarLengthArray and fix CMake, that's
> fine by me.
>     Now working on smokeqt. If there are failures coming from smokegen,
> any new code will be sent to the same branch (Qt5.0.1).
>
>     Regards,
>     Dimitar Dobrev
>
> _______________________________________________
> Kde-bindings mailing list
> Kde-bindings at kde.org
> https://mail.kde.org/mailman/listinfo/kde-bindings
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-bindings/attachments/20130405/25945034/attachment.html>