basevideoencoder: invoke subclass start method at state change
authorMark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Thu, 24 Mar 2011 08:21:04 +0000 (09:21 +0100)
committerMark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Mon, 16 May 2011 18:32:20 +0000 (20:32 +0200)
commit4b5623d6879892cbe119041d78996bc8db3ded1a
tree9d6029e7207968979cd9fdec68ed8955a6858143
parentd567fa2834d7a6c772e1c74edba171e33dd9005a
basevideoencoder: invoke subclass start method at state change

While this changes order w.r.t. set_format, which is OK for unstable API,
it has following merits:
* symmetric w.r.t. stop at state change
* in line with other base class practice
* little benefit in invoking 2 subclass virtual methods (set_format and start)
  in immediate succession; all actions in the second could be done in the first
  whereas subclass has no chance to do anything 'global' at activation time
Moreover, current -bad subclass relevant methods either trivially commute
or are either trivially adjusted accordingly.
ext/dirac/gstdiracenc.cc
ext/schroedinger/gstschroenc.c
gst-libs/gst/video/gstbasevideoencoder.c