]> git.nihav.org Git - nihav.git/blobdiff - src/codecs/h263/rv20.rs
Rust2018: use new addressing scheme
[nihav.git] / src / codecs / h263 / rv20.rs
index 9c87df90158779f1569d70f6a8e489242c711367..11dbe2edb199f73390617e4dd82d67b35cb60a61 100644 (file)
@@ -1,6 +1,6 @@
-use io::bitreader::*;
-use io::codebook::*;
-use formats;
+use crate::io::bitreader::*;
+use crate::io::codebook::*;
+use crate::formats;
 use super::super::*;
 use super::*;
 use super::code::H263BlockDSP;
@@ -11,7 +11,9 @@ use super::data::*;
 struct Tables {
     intra_mcbpc_cb: Codebook<u8>,
     inter_mcbpc_cb: Codebook<u8>,
+    mbtype_b_cb:    Codebook<u8>,
     cbpy_cb:        Codebook<u8>,
+    cbpc_b_cb:      Codebook<u8>,
     rl_cb:          Codebook<H263RLSym>,
     aic_rl_cb:      Codebook<H263RLSym>,
     mv_cb:          Codebook<u8>,
@@ -53,6 +55,7 @@ struct RealVideo20BR<'a> {
 
 struct RV20SliceInfo {
     ftype:  Type,
+    seq:    u32,
     qscale: u8,
     mb_x:   usize,
     mb_y:   usize,
@@ -62,8 +65,8 @@ struct RV20SliceInfo {
 }
 
 impl RV20SliceInfo {
-    fn new(ftype: Type, qscale: u8, mb_x: usize, mb_y: usize, mb_pos: usize, w: usize, h: usize) -> Self {
-        RV20SliceInfo { ftype: ftype, qscale: qscale, mb_x: mb_x, mb_y: mb_y, mb_pos: mb_pos, w: w, h: h }
+    fn new(ftype: Type, seq: u32, qscale: u8, mb_x: usize, mb_y: usize, mb_pos: usize, w: usize, h: usize) -> Self {
+        RV20SliceInfo { ftype: ftype, seq: seq, qscale: qscale, mb_x: mb_x, mb_y: mb_y, mb_pos: mb_pos, w: w, h: h }
     }
 }
 
@@ -108,22 +111,27 @@ impl<'a> RealVideo20BR<'a> {
     }
 
 #[allow(unused_variables)]
-    fn decode_block(&mut self, sstate: &SliceState, quant: u8, intra: bool, coded: bool, blk: &mut [i16; 64], plane_no: usize) -> DecoderResult<()> {
-        let mut br = &mut self.br;
+    fn decode_block(&mut self, sstate: &SliceState, quant: u8, intra: bool, coded: bool, blk: &mut [i16; 64], plane_no: usize, acpred: ACPredMode) -> DecoderResult<()> {
+        let br = &mut self.br;
         let mut idx = 0;
         if !sstate.is_iframe && intra {
-            let mut dc = br.read(8).unwrap() as i16;
+            let mut dc = br.read(8)? as i16;
             if dc == 255 { dc = 128; }
             blk[0] = dc << 3;
             idx = 1;
         }
         if !coded { return Ok(()); }
+        let scan = match acpred {
+                    ACPredMode::Hor => H263_SCAN_V,
+                    ACPredMode::Ver => H263_SCAN_H,
+                    _               => H263_ZIGZAG,
+                };
 
         let rl_cb = if sstate.is_iframe { &self.tables.aic_rl_cb } else { &self.tables.rl_cb };
-        let q_add = if quant == 0 { 0i16 } else { ((quant - 1) | 1) as i16 };
-        let q = (quant * 2) as i16;
+        let q_add = if quant == 0 || sstate.is_iframe { 0i16 } else { ((quant - 1) | 1) as i16 };
+        let q = if plane_no == 0 { (quant * 2) as i16 } else { H263_CHROMA_QUANT[quant as usize] as i16 };
         while idx < 64 {
-            let code = br.read_cb(rl_cb).unwrap();
+            let code = br.read_cb(rl_cb)?;
             let run;
             let mut level;
             let last;
@@ -131,15 +139,15 @@ impl<'a> RealVideo20BR<'a> {
                 run   = code.get_run();
                 level = code.get_level();
                 last  = code.is_last();
-                if br.read_bool().unwrap() { level = -level; }
+                if br.read_bool()? { level = -level; }
                 level = (level * q) + q_add;
             } else {
-                last  = br.read_bool().unwrap();
-                run   = br.read(6).unwrap() as u8;
-                level = br.read_s(8).unwrap() as i16;
+                last  = br.read_bool()?;
+                run   = br.read(6)? as u8;
+                level = br.read_s(8)? as i16;
                 if level == -128 {
-                    let low = br.read(5).unwrap() as i16;
-                    let top = br.read_s(6).unwrap() as i16;
+                    let low = br.read(5)? as i16;
+                    let top = br.read_s(6)? as i16;
                     level = (top << 5) | low;
                 }
                 level = (level * q) + q_add;
@@ -148,7 +156,7 @@ impl<'a> RealVideo20BR<'a> {
             }
             idx += run;
             validate!(idx < 64);
-            let oidx = H263_ZIGZAG[idx as usize];
+            let oidx = scan[idx as usize];
             blk[oidx] = level;
             idx += 1;
             if last { break; }
@@ -158,9 +166,9 @@ impl<'a> RealVideo20BR<'a> {
 }
 
 fn decode_mv_component(br: &mut BitReader, mv_cb: &Codebook<u8>) -> DecoderResult<i16> {
-    let code = br.read_cb(mv_cb).unwrap() as i16;
+    let code = br.read_cb(mv_cb)? as i16;
     if code == 0 { return Ok(0) }
-    if !br.read_bool().unwrap() {
+    if !br.read_bool()? {
         Ok(code)
     } else {
         Ok(-code)
@@ -168,45 +176,52 @@ fn decode_mv_component(br: &mut BitReader, mv_cb: &Codebook<u8>) -> DecoderResul
 }
 
 fn decode_mv(br: &mut BitReader, mv_cb: &Codebook<u8>) -> DecoderResult<MV> {
-    let xval = decode_mv_component(br, mv_cb).unwrap();
-    let yval = decode_mv_component(br, mv_cb).unwrap();
-//println!("  MV {},{} @ {}", xval, yval, br.tell());
+    let xval = decode_mv_component(br, mv_cb)?;
+    let yval = decode_mv_component(br, mv_cb)?;
     Ok(MV::new(xval, yval))
 }
 
+fn read_dquant(br: &mut BitReader, q: u8) -> DecoderResult<u8> {
+    if br.read_bool()? {
+        Ok(H263_MODIFIED_QUANT[br.read(1)? as usize][q as usize])
+    } else {
+        Ok(br.read(5)? as u8)
+    }
+}
+
 impl<'a> BlockDecoder for RealVideo20BR<'a> {
 
 #[allow(unused_variables)]
     fn decode_pichdr(&mut self) -> DecoderResult<PicInfo> {
         self.slice_no = 0;
-println!("decoding picture header size {}", if self.num_slices > 1 { self.slice_off[1] } else { ((self.br.tell() as u32) + (self.br.left() as u32))/8 });
-        let shdr = self.read_slice_header().unwrap();
-println!("slice ends @ {}\n", self.br.tell());
+        let shdr = self.read_slice_header()?;
 //        self.slice_no += 1;
         validate!((shdr.mb_x == 0) && (shdr.mb_y == 0));
 /*        let mb_count;
         if self.slice_no < self.num_slices {
             let pos = self.br.tell();
-            let shdr2 = self.read_slice_header().unwrap();
+            let shdr2 = self.read_slice_header()?;
             self.br.seek(pos as u32)?;
             mb_count = shdr2.mb_pos - shdr.mb_pos;
         } else {
             mb_count = self.mb_w * self.mb_h;
         }*/
 
-        let picinfo = PicInfo::new(shdr.w, shdr.h, shdr.ftype, false, false, shdr.qscale, 0, None, None);
+        let plusinfo = Some(PlusInfo::new(shdr.ftype == Type::I, false, false, false));
+        let picinfo = PicInfo::new(shdr.w, shdr.h, shdr.ftype, MVMode::Long, false, false, shdr.qscale, shdr.seq as u16, None, plusinfo);
         Ok(picinfo)
     }
 
     #[allow(unused_variables)]
     fn decode_slice_header(&mut self, info: &PicInfo) -> DecoderResult<SliceInfo> {
-//println!("read slice {} header", self.slice_no);
-        let shdr = self.read_slice_header().unwrap();
+        let shdr = self.read_slice_header()?;
         self.slice_no += 1;
         let mb_count;
         if self.slice_no < self.num_slices {
-            let shdr2 = self.read_slice_header().unwrap();
+            let pos = self.br.tell();
+            let shdr2 = self.read_slice_header()?;
             mb_count = shdr2.mb_pos - shdr.mb_pos;
+            self.br.seek(pos as u32)?;
         } else {
             mb_count = self.mb_w * self.mb_h - shdr.mb_pos;
         }
@@ -215,92 +230,126 @@ println!("slice ends @ {}\n", self.br.tell());
         Ok(ret)
     }
 
-    fn decode_block_header(&mut self, info: &PicInfo, slice: &SliceInfo, sstate: &SliceState) -> DecoderResult<BlockInfo> {
-        let mut br = &mut self.br;
-        let mut q = slice.get_quant();
+    fn decode_block_header(&mut self, info: &PicInfo, _slice: &SliceInfo, sstate: &SliceState) -> DecoderResult<BlockInfo> {
+        let br = &mut self.br;
+        let mut q = sstate.quant;
         match info.get_mode() {
             Type::I => {
-                    let mut cbpc = br.read_cb(&self.tables.intra_mcbpc_cb).unwrap();
-                    while cbpc == 8 { cbpc = br.read_cb(&self.tables.intra_mcbpc_cb).unwrap(); }
-if sstate.is_iframe {
-let acpred = br.read_bool()?;
-println!("   acp {} @ {}", acpred, br.tell());
-if acpred {
-    br.skip(1)?;//pred direction
-}
-}
-                    let cbpy = br.read_cb(&self.tables.cbpy_cb).unwrap();
+                    let mut cbpc = br.read_cb(&self.tables.intra_mcbpc_cb)?;
+                    while cbpc == 8 { cbpc = br.read_cb(&self.tables.intra_mcbpc_cb)?; }
+                    let mut acpred = ACPredMode::None;
+                    if let Some(ref pi) = info.plusinfo {
+                        if pi.aic {
+                            let acpp = br.read_bool()?;
+                            acpred = ACPredMode::DC;
+                            if acpp {
+                                acpred = if br.read_bool()? { ACPredMode::Hor } else { ACPredMode::Ver };
+                            }
+                        }
+                    }
+                    let cbpy = br.read_cb(&self.tables.cbpy_cb)?;
                     let cbp = (cbpy << 2) | (cbpc & 3);
                     let dquant = (cbpc & 4) != 0;
                     if dquant {
-                        let idx = br.read(2).unwrap() as usize;
-                        q = ((q as i16) + (H263_DQUANT_TAB[idx] as i16)) as u8;
+                        q = read_dquant(br, q)?;
                     }
-println!(" MB {},{} CBP {:X} @ {}", sstate.mb_x, sstate.mb_y, cbp, br.tell());
-                    Ok(BlockInfo::new(Type::I, cbp, q))
+                    let mut binfo = BlockInfo::new(Type::I, cbp, q);
+                    binfo.set_acpred(acpred);
+                    Ok(binfo)
                 },
             Type::P => {
-                    if br.read_bool().unwrap() {
+                    if br.read_bool()? {
                         return Ok(BlockInfo::new(Type::Skip, 0, info.get_quant()));
                     }
-                    let mut cbpc = br.read_cb(&self.tables.inter_mcbpc_cb).unwrap();
-                    while cbpc == 20 { cbpc = br.read_cb(&self.tables.inter_mcbpc_cb).unwrap(); }
+                    let mut cbpc = br.read_cb(&self.tables.inter_mcbpc_cb)?;
+                    while cbpc == 20 { cbpc = br.read_cb(&self.tables.inter_mcbpc_cb)?; }
                     let is_intra = (cbpc & 0x04) != 0;
                     let dquant   = (cbpc & 0x08) != 0;
                     let is_4x4   = (cbpc & 0x10) != 0;
                     if is_intra {
-                        let cbpy = br.read_cb(&self.tables.cbpy_cb).unwrap();
+                        let cbpy = br.read_cb(&self.tables.cbpy_cb)?;
                         let cbp = (cbpy << 2) | (cbpc & 3);
                         if dquant {
-                            let idx = br.read(2).unwrap() as usize;
-                            q = ((q as i16) + (H263_DQUANT_TAB[idx] as i16)) as u8;
+                            q = read_dquant(br, q)?;
                         }
                         let binfo = BlockInfo::new(Type::I, cbp, q);
                         return Ok(binfo);
                     }
 
-                    let mut cbpy = br.read_cb(&self.tables.cbpy_cb).unwrap();
+                    let mut cbpy = br.read_cb(&self.tables.cbpy_cb)?;
 //                    if /* !aiv && */(cbpc & 3) != 3 {
                         cbpy ^= 0xF;
 //                    }
                     let cbp = (cbpy << 2) | (cbpc & 3);
                     if dquant {
-                        let idx = br.read(2).unwrap() as usize;
-                        q = ((q as i16) + (H263_DQUANT_TAB[idx] as i16)) as u8;
+                        q = read_dquant(br, q)?;
                     }
-println!(" MB {}.{} cbp = {:X}", sstate.mb_x, sstate.mb_y, cbp);
                     let mut binfo = BlockInfo::new(Type::P, cbp, q);
                     if !is_4x4 {
-                        let mvec: [MV; 1] = [decode_mv(br, &self.tables.mv_cb).unwrap()];
+                        let mvec: [MV; 1] = [decode_mv(br, &self.tables.mv_cb)?];
                         binfo.set_mv(&mvec);
                     } else {
                         let mvec: [MV; 4] = [
-                                decode_mv(br, &self.tables.mv_cb).unwrap(),
-                                decode_mv(br, &self.tables.mv_cb).unwrap(),
-                                decode_mv(br, &self.tables.mv_cb).unwrap(),
-                                decode_mv(br, &self.tables.mv_cb).unwrap()
+                                decode_mv(br, &self.tables.mv_cb)?,
+                                decode_mv(br, &self.tables.mv_cb)?,
+                                decode_mv(br, &self.tables.mv_cb)?,
+                                decode_mv(br, &self.tables.mv_cb)?
                             ];
                         binfo.set_mv(&mvec);
                     }
                     Ok(binfo)
                 },
-            _ => { println!("wrong info mode"); Err(DecoderError::InvalidData) },
+            Type::B => { // B
+                    let mut mbtype = br.read_cb(&self.tables.mbtype_b_cb)? as usize;
+                    while mbtype == 14 { mbtype = br.read_cb(&self.tables.mbtype_b_cb)? as usize; }
+
+                    let is_coded  = (H263_MBTYPE_B_CAPS[mbtype] & H263_MBB_CAP_CODED) != 0;
+                    let is_intra  = (H263_MBTYPE_B_CAPS[mbtype] & H263_MBB_CAP_INTRA) != 0;
+                    let dquant    = (H263_MBTYPE_B_CAPS[mbtype] & H263_MBB_CAP_DQUANT) != 0;
+                    let is_fwd    = (H263_MBTYPE_B_CAPS[mbtype] & H263_MBB_CAP_FORWARD) != 0;
+                    let is_bwd    = (H263_MBTYPE_B_CAPS[mbtype] & H263_MBB_CAP_BACKWARD) != 0;
+
+                    let cbp = if is_coded {
+                            let cbpc = br.read_cb(&self.tables.cbpc_b_cb)?;
+                            let mut cbpy = br.read_cb(&self.tables.cbpy_cb)?;
+                            if !is_intra { cbpy ^= 0xF; }
+                            (cbpy << 2) | (cbpc & 3)
+                        } else { 0 };
+
+                    if dquant {
+                        q = read_dquant(br, q)?;
+                    }
+
+                    if is_intra {
+                        let binfo = BlockInfo::new(Type::I, cbp, q);
+                        return Ok(binfo);
+                    }
+
+                    let mut binfo = BlockInfo::new(Type::B, cbp, q);
+                    if is_fwd {
+                        let mvec: [MV; 1] = [decode_mv(br, &self.tables.mv_cb)?];
+                        binfo.set_mv(&mvec);
+                    }
+                    if is_bwd {
+                        let mvec: [MV; 1] = [decode_mv(br, &self.tables.mv_cb)?];
+                        binfo.set_b_mv(&mvec);
+                    }
+                    Ok(binfo)
+                },
+            _ => { unreachable!(); },
         }
     }
 
-    #[allow(unused_variables)]
     fn decode_block_intra(&mut self, info: &BlockInfo, sstate: &SliceState, quant: u8, no: usize, coded: bool, blk: &mut [i16; 64]) -> DecoderResult<()> {
-        self.decode_block(sstate, quant, true, coded, blk, if no < 4 { 0 } else { no - 3 })
+        self.decode_block(sstate, quant, true, coded, blk, if no < 4 { 0 } else { no - 3 }, info.get_acpred())
     }
 
     #[allow(unused_variables)]
     fn decode_block_inter(&mut self, info: &BlockInfo, sstate: &SliceState, quant: u8, no: usize, coded: bool, blk: &mut [i16; 64]) -> DecoderResult<()> {
-        self.decode_block(sstate, quant, false, coded, blk, if no < 4 { 0 } else { no - 3 })
+        self.decode_block(sstate, quant, false, coded, blk, if no < 4 { 0 } else { no - 3 }, ACPredMode::None)
     }
 
     fn is_slice_end(&mut self) -> bool { false }
-
-    fn is_gob(&mut self) -> bool { false }
 }
 
 impl<'a> RealVideo20BR<'a> {
@@ -308,39 +357,38 @@ impl<'a> RealVideo20BR<'a> {
     fn read_slice_header(&mut self) -> DecoderResult<RV20SliceInfo> {
         validate!(self.slice_no < self.num_slices);
 
-        let mut br = &mut self.br;
-        br.seek(self.slice_off[self.slice_no] * 8).unwrap();
-//println!(" slice at off {}", br.tell());
+        let br = &mut self.br;
+        br.seek(self.slice_off[self.slice_no] * 8)?;
 
-        let frm_type    = br.read(2).unwrap();
+        let frm_type    = br.read(2)?;
         let ftype = match frm_type {
                 0 | 1 => { Type::I },
                 2     => { Type::P },
-                _     => { Type::Skip },
+                _     => { Type::B },
             };
 
-        let marker      = br.read(1).unwrap();
+        let marker      = br.read(1)?;
         validate!(marker == 0);
-        let qscale      = br.read(5).unwrap() as u8;
+        let qscale      = br.read(5)? as u8;
         validate!(qscale > 0);
         if self.minor_ver >= 2 {
-            br.skip(1).unwrap(); // loop filter
+            br.skip(1)?; // loop filter
         }
         let seq = if self.minor_ver <= 1 {
-                br.read(8).unwrap()  << 7
+                br.read(8)?  << 8
             } else {
-                br.read(13).unwrap() << 2
+                br.read(13)? << 3
             };
         let w;
         let h;
         if self.rpr.present {
-            let rpr = br.read(self.rpr.bits).unwrap() as usize;
+            let rpr = br.read(self.rpr.bits)? as usize;
             if rpr == 0 {
                 w = self.w;
                 h = self.h;
             } else {
-                w = self.rpr.widths[rpr];
-                h = self.rpr.heights[rpr];
+                w = self.rpr.widths[rpr - 1];
+                h = self.rpr.heights[rpr - 1];
                 validate!((w != 0) && (h != 0));
             }
         } else {
@@ -348,18 +396,17 @@ impl<'a> RealVideo20BR<'a> {
             h = self.h;
         }
 
-        let mb_pos = br.read(self.mb_pos_bits).unwrap() as usize;
+        let mb_pos = br.read(self.mb_pos_bits)? as usize;
         let mb_x = mb_pos % self.mb_w;
         let mb_y = mb_pos / self.mb_w;
 
-        br.skip(1).unwrap(); // no rounding
+        br.skip(1)?; // no rounding
 
         if (self.minor_ver <= 1) && (frm_type == 3) {
-            br.skip(5).unwrap();
+            br.skip(5)?;
         }
-println!("slice q {} mb {},{}", qscale, mb_x, mb_y);
 
-        Ok(RV20SliceInfo::new(ftype, qscale, mb_x, mb_y, mb_pos, w, h))
+        Ok(RV20SliceInfo::new(ftype, seq, qscale, mb_x, mb_y, mb_pos, w, h))
     }
 }
 
@@ -369,19 +416,25 @@ impl RealVideo20Decoder {
         let intra_mcbpc_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
         let mut coderead = H263ShortCodeReader::new(H263_INTER_MCBPC);
         let inter_mcbpc_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
+        let mut coderead = H263ShortCodeReader::new(H263_MBTYPE_B);
+        let mbtype_b_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
         let mut coderead = H263ShortCodeReader::new(H263_CBPY);
         let cbpy_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
+        let mut coderead = H263ShortCodeReader::new(H263_CBPC_B);
+        let cbpc_b_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
         let mut coderead = H263RLCodeReader::new(H263_RL_CODES);
         let rl_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
         let mut coderead = H263RLCodeReader::new(H263_RL_CODES_AIC);
         let aic_rl_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
         let mut coderead = H263ShortCodeReader::new(H263_MV);
         let mv_cb = Codebook::new(&mut coderead, CodebookMode::MSB).unwrap();
-        
+
         let tables = Tables {
             intra_mcbpc_cb: intra_mcbpc_cb,
             inter_mcbpc_cb: inter_mcbpc_cb,
+            mbtype_b_cb:    mbtype_b_cb,
             cbpy_cb:        cbpy_cb,
+            cbpc_b_cb:      cbpc_b_cb,
             rl_cb:          rl_cb,
             aic_rl_cb:      aic_rl_cb,
             mv_cb:          mv_cb,
@@ -389,7 +442,7 @@ impl RealVideo20Decoder {
 
         RealVideo20Decoder{
             info:           Rc::new(DUMMY_CODEC_INFO),
-            dec:            H263BaseDecoder::new(),
+            dec:            H263BaseDecoder::new_b_frames(false),
             tables:         tables,
             w:              0,
             h:              0,
@@ -418,7 +471,6 @@ impl NADecoder for RealVideo20Decoder {
             let maj_ver = ver >> 16;
             let min_ver = (ver >> 8) & 0xFF;
             let mic_ver = ver & 0xFF;
-println!("ver {:06X}", ver);
             validate!(maj_ver == 2);
             self.minor_ver = min_ver as u8;
             let rprb = src[1] & 7;
@@ -426,7 +478,7 @@ println!("ver {:06X}", ver);
                 self.rpr.present = false;
             } else {
                 self.rpr.present = true;
-                self.rpr.bits    = rprb as u8;
+                self.rpr.bits    = ((rprb >> 1) + 1) as u8;
                 for i in 4..(src.len()/2) {
                     self.rpr.widths [i - 4] = (src[i * 2]     as usize) * 4;
                     self.rpr.heights[i - 4] = (src[i * 2 + 1] as usize) * 4;
@@ -434,21 +486,19 @@ println!("ver {:06X}", ver);
             }
             Ok(())
         } else {
-println!(".unwrap().unwrap().unwrap()");
             Err(DecoderError::InvalidData)
         }
     }
     fn decode(&mut self, pkt: &NAPacket) -> DecoderResult<NAFrameRef> {
         let src = pkt.get_buffer();
 
-println!(" decode frame size {}, {} slices", src.len(), src[0]+1);
         let mut ibr = RealVideo20BR::new(&src, &self.tables, self.w, self.h, self.minor_ver, self.rpr);
 
-        let bufinfo = self.dec.parse_frame(&mut ibr, &self.bdsp).unwrap();
+        let bufinfo = self.dec.parse_frame(&mut ibr, &self.bdsp)?;
 
         let mut frm = NAFrame::new_from_pkt(pkt, self.info.clone(), bufinfo);
         frm.set_keyframe(self.dec.is_intra());
-        frm.set_frame_type(if self.dec.is_intra() { FrameType::I } else { FrameType::P });
+        frm.set_frame_type(self.dec.get_frame_type());
         Ok(Rc::new(RefCell::new(frm)))
     }
 }
@@ -470,9 +520,10 @@ pub fn get_decoder() -> Box<NADecoder> {
 
 #[cfg(test)]
 mod test {
-    use test::dec_video::test_file_decoding;
+    use crate::test::dec_video::test_file_decoding;
     #[test]
     fn test_rv20() {
-         test_file_decoding("realmedia", "assets/RV/rv20_cook_640x352_realproducer_plus_8.51.rm", /*None*/Some(160), true, false, Some("rv20"));
+        test_file_decoding("realmedia", "assets/RV/rv20_svt_atrc_640x352_realproducer_plus_8.51.rm", /*None*/Some(3000), true, false, None/*Some("rv20")*/);
+//        test_file_decoding("realmedia", "assets/RV/rv20_cook_640x352_realproducer_plus_8.51.rm", /*None*/Some(1000), true, false, Some("rv20"));
     }
 }