[Bug 748497] spandsp: Fails to build, uses a private field missing_samples
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Sun Jun 14 08:48:13 PDT 2015
https://bugzilla.gnome.org/show_bug.cgi?id=748497
Tim-Philipp Müller <t.i.m at zen.co.uk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |RESOLVED
Resolution|--- |FIXED
--- Comment #15 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Let's close this. I think the current code is good as it is, and the
risk/benefit ratio of changing it as suggested seems high.
Seeing that accessing that private member is only an optimisation *and* we
don't even know for sure if it's a legit optimisation/assumption to make, we
should probably just leave it as is. Besides, the performance impact of
gst_buffer_make_writable() is tiny, and skipping it is wrong anyway if we map
with READWRITE (what we should've done instead is to map as READ only I guess).
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list