[mythtv] [PATCH] Split capture and encoding functions

Geoffrey Hausheer ou401cru02 at sneakemail.com
Thu Jul 17 20:18:50 EDT 2003


On Thu, 17 Jul 2003 19:02:31 -0400, "Isaac Richards ijr-at-po.cwru.edu
|mythtv/1.0-Allow|" <e6kbhbk35r0t at sneakemail.com> said:
> On Thursday 17 July 2003 03:01 pm, Geoffrey Hausheer wrote:
> > This patch splits out the encoding sections of NuppelVideoRecorder into a
> > seperate class.  It initially will allow me to clean up the Reencoder a
> > bit (no hacks needed to disable the capture-related sections).  In the
> > longer term, it should allow mythtv to support Reencoding to any
> > file-format without mencoder, (as well as recording to any format that
> > can support the prerequisites for streaming data).
> 
> I think this is unnecessary.  It munges up the NVR class rather badly.
I certainly won't argue about the second point.  I wish there were a way
to fix it, but I think it would require a nearly complete rewrite of NVR.
 As for unnecessary, I suppose that's true since nuv recording already
works, and there is no other output format the the devs feel is viable,
nor does there seem to be one on the horizon.  However, I see no reason
that the capture outines and encoding routines should be tied together,
and it will certainly be a bit more inconvenient for the transcoder. 
Personally, I think that since mythtv already has support for native
.mpeg2 files, there is no reason why users shouldn't be able to choose to
record to them or to avi regardless of whether they have a
hardware-encoder or not (of course, today not many computers are capable
of doing realtime MPEG2 encoding at high res without hardware).  I would
prefer not to need to transcode in order to create a portable video, and
I can only achieve that today by using the PVR-250.

Anyhow, that is my argument.  In the end, you have the final say, and by
telling me now, you saved me wasting my weekend working on it.  

.Geoff
-- 
  Geoffrey Hausheer
  XXXXXXXXXXXXXXXXXXXXX


More information about the mythtv-dev mailing list