[compiz] Features for image loader plugins

[Patrick Niklaus]

Hi,

I just had a look at the old beryl plugin tree and found out that it
originally was "imagemime:image/png" (for using mime types as
identification) and "imageext:png" (of file extension identification).

While porting the features to metadata "imagemime" has become
"imageext" (probably just a little c&p mistake).

So actually it makes sense to have a mime-type and extension identification.

I attached an updated patch which should fix the typo.

Cheers,
Patrick

2007/7/12, David Reveman <davidr at novell.com>:
> On Wed, 2007-06-13 at 19:01 +0200, Patrick Niklaus wrote:
> > I just found out that the old specification for this were
> > "imageext:..." so I attached a new patch to use this instead. Also I
> > fixed my indention to use compiz style.
> >
> > Regards,
> > Patrick
> >
> > 2007/6/13, Patrick Niklaus <patrick.niklaus at googlemail.com>:
> > > Hi,
> > > I just noticed that the current image loader plugins such as PNG, SVG
> > > and JPEG don't provide a feature as they used to do. This is needed
> > > for conflict checking and feature testing (e.g. by the settings
> > > manager). If I remember correctly this plugins had features before the
> > > metadata port so I did a patch to add them again.
> > > What do you think about that?
> > >
> > > Regards,
> > > Patrick
> > >
> > >
>
> Why both
> +       <feature>imageext:png</feature>
> and
> +       <feature>imageext:image/png</feature>
> ?
> Backward compatibility? In which case it might make sense to abandon the
> old way as the configuration utility needs to be updated to fetch the
> feature from the metadata anyhow.
>
> -David
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: img-feature.patch
Type: text/x-patch
Size: 675 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/compiz/attachments/20070712/1291e160/attachment.bin