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

List:       kde-commits
Subject:    Re: KDE/kdebase/runtime/plasma/scriptengines/javascript/common
From:       David Faure <faure () kde ! org>
Date:       2010-12-10 22:42:16
Message-ID: 201012102342.16959.faure () kde ! org
[Download RAW message or body]

On Friday 10 December 2010, Aaron J. Seigo wrote:
> ---
> trunk/KDE/kdebase/runtime/plasma/scriptengines/javascript/common/scriptenv
> .cpp #1205332:1205333 @@ -214,7 +214,8 @@
>      QScriptValue v = context->argument(0);
>      KUrl url = v.isString() ? KUrl(v.toString()) : qscriptvalue_cast<KUrl>(v); 
>      if (url.isValid()) {
> -        return KRun::runUrl(url, KMimeType::findByUrl(url)->name(), 0);
> +        KMimeType::Ptr mimetype = KMimeType::findByUrl(url, 0, false, false); 
> +        new KRun(url, 0);
>      }
>  
>      return false;

Well, you can remove the KMimeType line altogether now.

-- 
David Faure, faure@kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).
[prev in list] [next in list] [prev in thread] [next in thread]