[Bug 733233] chromaprint: notify fingerprint also via property notify

Wed Jul 16 08:06:06 PDT 2014

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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk
            Summary|gstchromaprint doesn't      |chromaprint: notify
                   |notify on fingerprint       |fingerprint also via
                   |property change             |property notify
           Severity|normal                      |enhancement

--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2014-07-16 15:06:01 UTC ---
I would just emit the notify by name, the pspec thing is only really needed if
this is something you might be doing a *lot* and performance is important. Not
sure that is the case here.

The reason we tend to not notify such things via properties is that the
notification will come from a streaming thread, which causes all kinds of
problems.

-- 
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.