[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