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 657033 - kate: emit timed metadata as tags
kate: emit timed metadata as tags
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal enhancement
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-08-21 19:49 UTC by ogg.k.ogg.k
Modified: 2011-08-23 08:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
emit timed metadata as tags (2.66 KB, patch)
2011-08-21 19:49 UTC, ogg.k.ogg.k
none Details | Review
use memchr instead of custom loop (2.66 KB, patch)
2011-08-22 07:05 UTC, ogg.k.ogg.k
committed Details | Review

Description ogg.k.ogg.k 2011-08-21 19:49:40 UTC
Created attachment 194328 [details] [review]
emit timed metadata as tags

Metadata attached to other objects that may be used in this
event are not passed, as is binary metadata.
Comment 1 David Schleef 2011-08-22 02:15:49 UTC
+  return !*value;

This appears to read past the end of the array.  If this is correct, it is counterintuitive, so should be commented.

Also, please use standard string functions whenever possible.
Comment 2 ogg.k.ogg.k 2011-08-22 07:05:33 UTC
Created attachment 194345 [details] [review]
use memchr instead of custom loop
Comment 3 ogg.k.ogg.k 2011-08-22 07:06:43 UTC
Ah, it would indeed for a buffer entirely made of zeroes, as this would pass the validate call.
Now using memchr, which I didn't think about before, and reorder tests to move the most expensive last.

Though I'm having second thoughts on this. What I really want is buffer metadata, which aren't in 0.10, so tags are a fallback way of exposing this, but I'll think some more about it.

(Hmm, it seems comments are dumped when also attaching a file, reposting)
Comment 4 Sebastian Dröge (slomo) 2011-08-23 08:35:15 UTC
commit d1a6ca22033060ae22f58d08b2eef7586868d92f
Author: Vincent Penquerc'h <ogg.k.ogg.k@googlemail.com>
Date:   Sun Aug 21 20:36:07 2011 +0100

    kate: pass on timed event metadata as tags when we can
    
    Metadata attached to other objects that may be used in this
    event are not passed, as is binary metadata.