Add AVIDemux Project export capability to mythcommflag

[robert.mcnamara@…]

This is a patch to add two new command line options to mythcommflag. They are:

--getskiplist-avidemux

and

--getcutlist-avidemux

These options, combined with --outputfile somefile, will export the cut and skip lists as AVIDemux compatible project files. This is useful for a number of reasons, principally for the lossless commercial cutting of formats like H.264 and XVid produced by several new hardware encoders, most notably the Hauppauge HD-PVR. This simplifies the workflow by generating a working, correct AVIDemux project file, requiring only a "File->Save" in AVIDemux to produce an edited copy of the recording. It can also be accomplished in a user job with a single command line.

Known defects include:

The last frame of the project is always set to frame 1,000,000 since, try as I might, I was unable to get nvp->GetTotalFrameCount?() to return any value. If someone knows the NuppelVideoPlayer? options better than I do, it is very likely a simple fix. To work around the problem I just set the last frame number to something obscenely high. AVIDemux just amends that number to the actual last frame when it opens the project.

[robert.mcnamara@…]

Patch adds MythCommflag? AVIDemux export

[robert.mcnamara@…]

Slight Improvement (AVI is a better container choice for HD-PVR)

[cpinkham]

Closing ticket after IRC discussion with Robert. A better solution is being worked on and it would probably be best if things like this were in a contrib script rather than hardcoded in the source. Perhaps even a version that took an optional format string to describe the output format of the information rather than hardcoding every desired format.

[databubble]

I applied mythcommflag-avidemux.2.diff to trunk 20741. Seemed to apply fine, but compile fails:

main.cpp: In function ‘int GetAviDemux?(QString, QString, QString, const QString&)’: main.cpp:317: error: no matching function for call to ‘NuppelVideoPlayer::NuppelVideoPlayer?(const char [16], ProgramInfo?*&)’ ../../libs/libmythtv/NuppelVideoPlayer.h:109: note: candidates are: NuppelVideoPlayer::NuppelVideoPlayer?() ../../libs/libmythtv/NuppelVideoPlayer.h:105: note: NuppelVideoPlayer::NuppelVideoPlayer?(const NuppelVideoPlayer?&)

main.cpp:355: warning: suggest parentheses around ‘&&’ within ‘

’

main.cpp: In function ‘int main(int, char)’: main.cpp:1382: warning: ignoring return value of ‘int nice(int)’, declared with attribute warn_unused_result main.cpp:1403: warning: ignoring return value of ‘int nice(int)’, declared with attribute warn_unused_result make: * [main.o] Error 1

Any hints?

[robert.mcnamara@…]

This ticket is closed on my request, as there are better, cleaner ways of handling this. The patch is not being kept current against trunk any more.