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

List:       kde-commits
Subject:    Re: KDE/kdelibs/kio/kio
From:       Clarence Dang <dang () kde ! org>
Date:       2007-04-30 23:45:02
Message-ID: 200705010945.04319.dang () kde ! org
[Download RAW message or body]

From the record, I mean KDE 3.5, not 3.3.

On Sunday 29 April 2007 23:13, Clarence Dang wrote:
> SVN commit 659101 by dang:
>
> Don't claim to support empty mimetypes (e.g. kdelibs/kimgio/xv.desktop).
>
> Now, KImageIO::mimeTypes (KImageIO::Writing) does not return any empty
> mimetype like in KDE 3.3 (this makes 'kolourpaint doesnotexist.png',
> followed by CTRL+S, bring up the save filedialog again).
>
> The other changes are not in KDE 3.3 and might want to be backported.
>
>
>
>
>  M  +9 -2      kimageio.cpp
>
>
> --- trunk/KDE/kdelibs/kio/kio/kimageio.cpp #659100:659101
> @@ -51,6 +51,9 @@
>
>  QStringList KImageIO::typeForMime(const QString& mimeType)
>  {
> +    if ( mimeType.isEmpty() )
> +        return QStringList();
> +
>      KService::List services =
> KServiceTypeTrader::self()->query("QImageIOPlugins"); KService::Ptr
> service;
>      foreach(service, services) {
> @@ -70,7 +73,9 @@
>          if ( (service->property("X-KDE-Read").toBool() && mode == Reading)
> || (service->property("X-KDE-Write").toBool() && mode == Writing ) ) {
>
> -            mimeList.append(
> service->property("X-KDE-MimeType").toString() ); +            const
> QString mime = service->property("X-KDE-MimeType").toString(); +           
> if ( !mime.isEmpty() )
> +                mimeList.append( mime );
>          }
>      }
>
> @@ -87,7 +92,6 @@
>               (service->property("X-KDE-Write").toBool() && mode == Writing
> ) ) {
>
>              imagetypes +=
> service->property("X-KDE-ImageFormat").toStringList(); -
>          }
>      }
>      return imagetypes;
> @@ -95,6 +99,9 @@
>
>  bool KImageIO::isSupported( const QString& mimeType, Mode mode )
>  {
> +    if (mimeType.isEmpty() )
> +        return false;
> +
>      KService::List services =
> KServiceTypeTrader::self()->query("QImageIOPlugins"); KService::Ptr
> service;
>      foreach(service, services) {


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

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