[Bug 769717] h265parser: refactoring some methods to be more like h264parser

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #4 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to m][sko from comment #3)
> 1. I am aware of it but h264parse do same( ignore return value in same
> situations)

Yes, I'm aware of that. It seems wrong in both, that's what I meant above :)
Maybe another patch to fix that for h264parse and do it right from the
beginning in h265parse?

> 2.
> So should I add something like this from h264parse:
> https://bug732203.bugzilla-attachments.gnome.org/attachment.cgi?id=279218
> 
> GST_H264_PARSE_STATE_VALID
> (h264parse,GST_H264_PARSE_STATE_VALID_PICTURE_HEADERS))

Keeping them more similar is certainly a good thing, yes.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.