Update libgifski

gifski-api/Cargo.toml

@@ -10,7 +10,7 @@ license = "AGPL-3.0+"
 name = "gifski"
 readme = "README.md"
 repository = "https://github.com/ImageOptim/gifski"
-version = "0.10.1"
+version = "0.10.2"
 autobins = false
 edition = "2018"
 
@@ -28,7 +28,6 @@ imagequant = "2.11.9"
 imgref = "1.3.5"
 lodepng = "2.1.5"
 pbr = "1.0.0"
-rayon = "1.0.3"
 resize = "0.3.0"
 rgb = "0.8.10"
 wild = "2.0"

gifski-api/gifski.h

@@ -109,8 +109,8 @@ gifski *gifski_new(const GifskiSettings *settings);
  * You can add frames in any order, and they will be sorted by their `frame_number`.
  *
  * Presentation timestamp (PTS) is time in seconds, since start of the file, when this frame is to be displayed.
- * For a 20fps video it could be `frame_number/20.0`.
- * First frame must have PTS=0, and frames must not be more than 1 second apart.
+ * For a 20fps video it could be `frame_number/20.0`. First frame must have PTS=0.
+ * Frames with duplicate or out-of-order PTS will be skipped.
  *
  * Returns 0 (`GIFSKI_OK`) on success, and non-0 `GIFSKI_*` constant on error.
  */
@@ -129,8 +129,8 @@ GifskiError gifski_add_frame_png_file(gifski *handle,
  * You can add frames in any order, and they will be sorted by their `frame_number`.
  *
  * Presentation timestamp (PTS) is time in seconds, since start of the file, when this frame is to be displayed.
- * For a 20fps video it could be `frame_number/20.0`.
- * First frame must have PTS=0, and frames must not be more than 1 second apart.
+ * For a 20fps video it could be `frame_number/20.0`. First frame must have PTS=0.
+ * Frames with duplicate or out-of-order PTS will be skipped.
  *
  * Returns 0 (`GIFSKI_OK`) on success, and non-0 `GIFSKI_*` constant on error.
  */

gifski-api/src/bin/ffmpeg_source.rs

@@ -22,7 +22,7 @@ impl Source for FfmpegDecoder {
 }
 
 impl FfmpegDecoder {
-    pub fn new(path: &Path, fps: usize) -> BinResult<Self> {
+    pub fn new(path: &Path, fps: f32) -> BinResult<Self> {
         ffmpeg::init().chain_err(|| "Unable to initialize ffmpeg")?;
         let input_context = ffmpeg::format::input(&path)
             .chain_err(|| format!("Unable to open video file {}", path.display()))?;

gifski-api/src/bin/gifski.rs

@@ -115,7 +115,7 @@ fn bin_main() -> BinResult<()> {
         fast: matches.is_present("fast"),
     };
     let quiet = matches.is_present("quiet");
-    let fps: usize = matches.value_of("fps").ok_or("Missing fps")?.parse().chain_err(|| "FPS must be a number")?;
+    let fps: f32 = matches.value_of("fps").ok_or("Missing fps")?.parse().chain_err(|| "FPS must be a number")?;
 
     if settings.quality < 20 {
         if settings.quality < 1 {
@@ -183,12 +183,12 @@ fn parse_opt<T: ::std::str::FromStr<Err = ::std::num::ParseIntError>>(s: Option<
 }
 
 #[cfg(feature = "video")]
-fn get_video_decoder(path: &Path, fps: usize) -> BinResult<Box<dyn Source + Send>> {
+fn get_video_decoder(path: &Path, fps: f32) -> BinResult<Box<dyn Source + Send>> {
     Ok(Box::new(ffmpeg_source::FfmpegDecoder::new(path, fps)?))
 }
 
 #[cfg(not(feature = "video"))]
-fn get_video_decoder(_: &Path, _fps: usize) -> BinResult<Box<dyn Source + Send>> {
+fn get_video_decoder(_: &Path, _fps: f32) -> BinResult<Box<dyn Source + Send>> {
     Err(r"Video support is permanently disabled in this executable.
 
 To enable video decoding you need to recompile gifski from source with:

gifski-api/src/bin/png.rs

@@ -5,11 +5,11 @@ use std::path::PathBuf;
 
 pub struct Lodecoder {
     frames: Vec<PathBuf>,
-    fps: usize,
+    fps: f32,
 }
 
 impl Lodecoder {
-    pub fn new(frames: Vec<PathBuf>, fps: usize) -> Self {
+    pub fn new(frames: Vec<PathBuf>, fps: f32) -> Self {
         Self { frames, fps }
     }
 }