[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    playground/base/nepomuk-kde/scribo/plugins/opencalais
From:       Sebastian Trueg <sebastian () trueg ! de>
Date:       2009-10-26 18:41:42
Message-ID: 1256582502.682500.19193.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 1040725 by trueg:

* Use deletLater to delete the network reply as requested by the documentation -> \
                less crashing
* Do not use the sesame2 backend but redland. This is a little slower but does not \
crash.

CCMAIL: slauriere@mandriva.com


 M  +5 -5      lookupjob.cpp  


--- trunk/playground/base/nepomuk-kde/scribo/plugins/opencalais/lookupjob.cpp \
#1040724:1040725 @@ -60,10 +60,10 @@
 {
     delete resultModel;
 
-    // sesame2 is waaaay faster then redland which is the default in Soprano
-    if ( const Soprano::Backend* b = \
                Soprano::PluginManager::instance()->discoverBackendByName( "sesame2" \
                ) )
-        resultModel = b->createModel( Soprano::BackendSettings() << \
                Soprano::BackendSetting( Soprano::BackendOptionStorageMemory ) );
-    else
+    // sesame2 is waaaay faster then redland which is the default in Soprano but it \
crashes sometimes. So that needs fixing first. +//     if ( const Soprano::Backend* b \
= Soprano::PluginManager::instance()->discoverBackendByName( "sesame2" ) ) +//        \
resultModel = b->createModel( Soprano::BackendSettings() << Soprano::BackendSetting( \
Soprano::BackendOptionStorageMemory ) ); +//     else
         resultModel = Soprano::createModel( Soprano::BackendSettings() << \
Soprano::BackendSetting( Soprano::BackendOptionStorageMemory ) );  
     return resultModel;
@@ -160,7 +160,7 @@
     // FIXME: error handling
     QByteArray data = d->lastReply->readAll();
 //    kDebug() << d->lastReply->error() << d->lastReply->rawHeaderList() << data;
-    delete d->lastReply;
+    d->lastReply->deleteLater();
 
     if ( !d->createModel() ) {
         setErrorText( i18n( "Failed to create Soprano memory model. Most likely the \
installation misses Soprano backend plugins" ) );


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic