factor out keyframe interval option
authorKostya Shishkov <kostya.shishkov@gmail.com>
Sat, 6 Jun 2020 07:40:26 +0000 (09:40 +0200)
committerKostya Shishkov <kostya.shishkov@gmail.com>
Sat, 6 Jun 2020 07:40:26 +0000 (09:40 +0200)
nihav-commonfmt/src/codecs/cinepakenc.rs
nihav-core/src/options.rs

index 4271e1a7d1575cb13a174a724f8017c181f6f40c..ce50eee63d3899cd36c0958ef05585923e0be977 100644 (file)
@@ -984,7 +984,7 @@ impl NAEncoder for CinepakEncoder {
 
 const ENCODER_OPTS: &[NAOptionDefinition] = &[
     NAOptionDefinition {
-        name: "key_int", description: "Keyframe interval (0 - automatic)",
+        name: KEYFRAME_OPTION, description: KEYFRAME_OPTION_DESC,
         opt_type: NAOptionDefinitionType::Int(Some(0), Some(128)) },
     NAOptionDefinition {
         name: "nstrips", description: "Number of strips per frame (0 - automatic)",
@@ -1002,7 +1002,7 @@ println!("option {}", option.name);
             for opt_def in ENCODER_OPTS.iter() {
                 if opt_def.check(option).is_ok() {
                     match option.name {
-                        "key_int" => {
+                        KEYFRAME_OPTION => {
                             if let NAValue::Int(intval) = option.value {
                                 self.key_int = intval as u8;
                             }
index c2f2d4d0ab2971502a0014cc744962e489001b78..846a948701f2a1d044541ed84a1713c14b900fc6 100644 (file)
@@ -8,6 +8,11 @@
 use std::sync::Arc;
 use std::fmt;
 
+/// Common name for keyframe interval option.
+pub const KEYFRAME_OPTION: &'static str = "key_int";
+/// Common description for keyframe interval option.
+pub const KEYFRAME_OPTION_DESC: &'static str = "Keyframe interval (0 - automatic)";
+
 /// A list specifying option parsing and validating errors.
 #[derive(Clone,Copy,Debug,PartialEq)]
 pub enum OptionError {