[Bug 752613] qtdemux: unsigned raw 16 bit PCM audio not working

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Jul 23 13:38:44 PDT 2015


https://bugzilla.gnome.org/show_bug.cgi?id=752613

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #307731|none                        |needs-work
             status|                            |

--- Comment #11 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 307731
  --> https://bugzilla.gnome.org/attachment.cgi?id=307731
The above is the diff patch code.

Thanks for the patch and the sample files.

I can confirm the patch works.

Please could you update the patch as follows:

 - provide a patch in 'git format-patch' format with author and commit message
set and bugzilla url to this ticket in the commit message

 - there seem to be spaces and/or tabs at the end of some lines (they show up
in red when you do 'git diff' for uncommitted changes or 'git show' for the
last commit). Please remove those.

 - please ensure correct indentation (spaces only, no tabs please). The easiest
way is to run gst-indent on the code (but see next item)

 - I think in the NONE/raw case, we should just check for
stream->bytes_per_packet == 1 and use the current code, and otherwise just fall
through to the 'twos' case, which does what we want already.

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