After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 652742 - Update comment about the default pipeline
Update comment about the default pipeline
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: drago01
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-06-16 14:38 UTC by Raul Gutierrez Segales
Modified: 2015-02-27 16:37 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Update comment about the pipeline used (1.08 KB, patch)
2011-06-16 14:38 UTC, Raul Gutierrez Segales 		committed Details | Review

Description Raul Gutierrez Segales 2011-06-16 14:38:19 UTC 
Created attachment 190048 [details] [review]
Update comment about the pipeline used

Commit c6baee26 switch the recorder's output to webm but the comment for the default pipeline wasn't updated. The attached patch updates the comment.
Comment 1 drago01 2011-06-18 18:16:32 UTC 
Review of attachment 190048 [details] [review]:

Yeah the comment is outdated, it is probably worth it to investigate whether videorate is needed at all for webm/vp8. (As the comment says it was added due to specific shortcomings in the theora encoder).
Comment 2 Jean-François Fortin Tam 2012-10-25 21:04:00 UTC 
Drago, could you commit or update the patch and close this bug report? If it's obsolete, it should be marked as such.
Comment 3 Florian Müllner 2012-10-25 21:19:57 UTC 
(In reply to comment #2)
> If it's obsolete, it should be marked as such.

It's not - the pipeline has changed again, the comment is still the same :-)
Comment 4 Jasper St. Pierre (not reading bugmail) 2014-01-13 00:17:57 UTC 
This is a really easy fix. drago01, did you ever investigate whether videorate was needed?
Comment 5 Florian Müllner 2015-02-27 16:37:38 UTC 
Attachment 190048 [details] pushed as 7d05890 - Update comment about the pipeline used

Finally getting this off the list with the videorate part removed entirely - it hasn't been in the pipeline since commit d4a26fbf4be668e8a ...