[Kde-bindings] smokegen: Qt 5 support added

[Dimitar Dobrev]

    Hi, Michael,

    Unfortunately I've been busy with other issues and have no updates. In fact, I think you've done further than I did because I never got smokeqt to compile (CMake could not find the Qt libs). However, just the last two days I decided to work on SMOKE for Qt 5 so I'm going to look into it. I'd be really grateful for any advice you may have on building smokeqt with Qt 5.

    Regards,
    Dimitar Dobrev



________________________________
 From: Michael Lawrence <lawrence.michael at gene.com>
To: Dimitar Dobrev <dpldobrev at yahoo.com>; KDE bindings for other programming languages <kde-bindings at kde.org> 
Sent: Friday, April 5, 2013 7:55 PM
Subject: Re: [Kde-bindings] smokegen: Qt 5 support added
 

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.
>    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/5442f17c/attachment.html>