Prepare for releaze

Author: jeroen

NEWS

@@ -1,3 +1,5 @@
 0.8.6
+  - Update gifski crate to 0.8.6
   - save_gif() now properly closes graphics device when expr errors
-  - safer C code to prevent segfaults on write errors
+  - More rubust C code to prevent segfaults on write errors
+  - File paths are now converted to UTF-8 as described in gifski.h

R/gifski.R

@@ -37,7 +37,7 @@ gifski <- function(png_files, gif_file = 'animation.gif', width = 800, height =
   delay <- as.integer(delay * 100)
   loop <- as.logical(loop)
   progress <- as.logical(progress)
-  .Call(R_png_to_gif, png_files, gif_file, width, height, delay, loop, progress)
+  .Call(R_png_to_gif, enc2utf8(png_files), enc2utf8(gif_file), width, height, delay, loop, progress)
 }
 
 #' @export

src/wrapper.c

@@ -10,9 +10,9 @@
 
 /* data to pass to encoder thread */
 typedef struct {
-  const char * path;
+  const char *path;
   GifskiError err;
-  gifski * g;
+  gifski *g;
 } gifski_encoder_thread_info;
 
 /* gifski_write() blocks until main thread calls gifski_end_adding_frames() */
@@ -34,7 +34,7 @@ SEXP R_png_to_gif(SEXP png_files, SEXP gif_file, SEXP width, SEXP height, SEXP d
   settings.once = !Rf_asLogical(loop);
 
   /* init in main thread */
-  gifski * g = gifski_new(&settings);
+  gifski *g = gifski_new(&settings);
 
   /* create encoder thread; TODO: maybe we can use multiple encoder threads? */
   pthread_t encoder_thread;