[Bug 733087] Add WebP Image encoder

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=733087
  GStreamer | gst-plugins-bad | unspecified

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #9 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-08-11 12:57:58 UTC ---
(In reply to comment #8)
> (In reply to comment #2)
> > Review of attachment 280545 [details] [details]:
> > 
> > Looks good, just some minor comments and questions
> 
> > @@ +398,3 @@
> > +
> > +  enc->webp_config.lossless = enc->lossless;
> > +  enc->webp_config.method = enc->speed;
> > 
> > Why do you call it speed, while the webp API calls it method?
> 
> WebpConfig.method is "quality/speed trade-off (0=fast, 6=slower-better)" . The
> word speed is more appropriate for user. The word "method" doesn't make much
> sense from users perspective IMHO.

Makes sense, just that we did the same for vp8enc and in the end it was more
confusing for users because on the WebM wiki these things were called different
:)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.