Matroska importer inserts empty frames

[roine@…]

AddTrack?() does not verify that a frame has a non-zero duration, and will try to insert frames with zero duration which QuickTime will reject with -2014 invalidDuration.

[dconrad]

I'm pretty sure that Matroska requires timecodes to be increasing, so if we try to insert a frame with zero duration, that probably means that we've screwed up calculating the duration. I know that we do so for formats with out-of-order frames (#30, which I should fix soon). I think trac allows attachments up to ~200k, so if you have a file that isn't covered by #30 where this occurs in the first 200k, could you attach that or upload it somewhere else if it doesn't occur until later in the file?

I'll add a return so that we don't continue calling functions as though it were added, but is there any reason to specifically check for a 0 duration beforehand rather than bail on a -2014 return?

[dconrad]

(In [328]) Bail on error adding sample in AddFrame?(). Refs #102

[tick]

Did [328] fix this?

[tick]

Move back to 1.0 if there are any objections