ruby-zstds 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 385a6f6c852fb6364739a194c7093286f59832078c9f934828c7219c74b08208
-  data.tar.gz: c65081f337c79034e3bba1cd0ef852988bd81671cd4ffe962a249d2d999c4226
+  metadata.gz: e2279b518a90a526dd82247626f5969d01f6786de10b91802c5b6b7f3391ac26
+  data.tar.gz: 854b6e8d08c7cc2fa464e3acc25cf45ea6ff204809b5b2cb08cea94217e34b1f
 SHA512:
-  metadata.gz: cc1903ed422256c4e1bc432b19a77211fde31ebaf76eff5c191aaf74e7dfa71d744266b45e57e21bb64a40c0f3a982c09ee5da1134e3d5ff0c957b588cfb6e47
-  data.tar.gz: 18f7ce7cfea451b951544491cc7fdc79891f220c8cfb7a6b7cb99a7ae9f99c4154c2f173908a673c70ad2e9d741ddbbd4d26fb73f254d0284730d720654d8cb7
+  metadata.gz: 68781c24292733752c2e59022d71c31d15d73e73f7b5297374e720fc88e06100e1513772304b7591f16652bfef38b4991b9128949355fd0746bbae9d671eff67
+  data.tar.gz: dab52302d4ce11b2392d7578fd824c4e4b725c3a7d7c853454d05b7291eeb7a93b024fce8c8d475ea965202bb7795db9a3d59764a5aea012fea49d712a59ae4e

data/README.md

@@ -306,7 +306,7 @@ File maintains both source and destination buffers, it accepts both `source_buff
 
 ## Stream::Writer
 
-Its behaviour is similar to builtin [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipWriter.html).
+Its behaviour is similar to builtin [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipWriter.html).
 
 Writer maintains destination buffer only, so it accepts `destination_buffer_length` option only.
 
@@ -344,7 +344,7 @@ Set another encodings, `nil` is just for compatibility with `IO`.
 #tell
 ```
 
-See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
+See [`IO`](https://ruby-doc.org/core-2.7.0/IO.html) docs.
 
 ```
 #write(*objects)
@@ -354,7 +354,7 @@ See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
 #closed?
 ```
 
-See [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipWriter.html) docs.
+See [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipWriter.html) docs.
 
 ```
 #write_nonblock(object, *options)
@@ -376,11 +376,11 @@ Behaviour is the same as `IO#write_nonblock` method.
 #puts(*objects)
 ```
 
-Typical helpers, see [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipWriter.html) docs.
+Typical helpers, see [`Zlib::GzipWriter`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipWriter.html) docs.
 
 ## Stream::Reader
 
-Its behaviour is similar to builtin [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipReader.html).
+Its behaviour is similar to builtin [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipReader.html).
 
 Reader maintains both source and destination buffers, it accepts both `source_buffer_length` and `destination_buffer_length` options.
 
@@ -415,7 +415,7 @@ Set another encodings.
 #tell
 ```
 
-See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
+See [`IO`](https://ruby-doc.org/core-2.7.0/IO.html) docs.
 
 ```
 #read(bytes_to_read = nil, out_buffer = nil)
@@ -425,14 +425,14 @@ See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
 #closed?
 ```
 
-See [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipReader.html) docs.
+See [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipReader.html) docs.
 
 ```
 #readpartial(bytes_to_read = nil, out_buffer = nil)
 #read_nonblock(bytes_to_read, out_buffer = nil, *options)
 ```
 
-See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
+See [`IO`](https://ruby-doc.org/core-2.7.0/IO.html) docs.
 
 ```
 #getbyte
@@ -455,7 +455,7 @@ See [`IO`](https://ruby-doc.org/core-2.6.1/IO.html) docs.
 #ungetline(line)
 ```
 
-Typical helpers, see [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.6.1/libdoc/zlib/rdoc/Zlib/GzipReader.html) docs.
+Typical helpers, see [`Zlib::GzipReader`](https://ruby-doc.org/stdlib-2.7.0/libdoc/zlib/rdoc/Zlib/GzipReader.html) docs.
 
 ## Dictionary
 
@@ -489,9 +489,11 @@ Read dictionary id from buffer.
 
 ## CI
 
-Travis and Appveyor CI uses [scripts/ci_test.sh](scripts/ci_test.sh) directly.
-Cirrus and Circle CI uses prebuilt [scripts/test-images](scripts/test-images).
-Cirrus CI uses amd64 image, Circle CI - i686.
+See universal test script [scripts/ci_test.sh](scripts/ci_test.sh) for CI.
+Please visit [scripts/test-images](scripts/test-images).
+You can run this test script using many native and cross images.
+
+Cirrus CI uses `x86_64-pc-linux-gnu` image, Circle CI - `x86_64-gentoo-linux-musl` image.
 
 ## License
 

data/ext/zstds_ext/common.h

@@ -13,4 +13,7 @@
 typedef size_t       zstds_result_t;
 typedef uint_fast8_t zstds_ext_result_t;
 
+typedef uint8_t      zstds_ext_byte_t;
+typedef uint_fast8_t zstds_ext_byte_fast_t;
+
 #endif // ZSTDS_EXT_COMMON_H

data/ext/zstds_ext/dictionary.c

@@ -3,14 +3,11 @@
 
 #include "zstds_ext/dictionary.h"
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <string.h>
 #include <zdict.h>
 
 #include "ruby.h"
 #include "zstds_ext/buffer.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/error.h"
 #include "zstds_ext/macro.h"
 #include "zstds_ext/option.h"
@@ -54,7 +51,7 @@ VALUE zstds_ext_train_dictionary_buffer(VALUE ZSTDS_EXT_UNUSED(self), VALUE samp
     zstds_ext_raise_error(ZSTDS_EXT_ERROR_ALLOCATE_FAILED);
   }
 
-  uint8_t* samples_buffer = malloc(samples_size);
+  zstds_ext_byte_t* samples_buffer = malloc(samples_size);
   if (samples_buffer == NULL) {
     zstds_ext_raise_error(ZSTDS_EXT_ERROR_ALLOCATE_FAILED);
   }

data/ext/zstds_ext/error.c

@@ -6,7 +6,6 @@
 #include <zstd_errors.h>
 
 #include "ruby.h"
-#include "zstds_ext/common.h"
 
 zstds_ext_result_t zstds_ext_get_error(ZSTD_ErrorCode error_code)
 {

data/ext/zstds_ext/io.c

@@ -3,14 +3,11 @@
 
 #include "ruby/io.h"
 
-#include <stdint.h>
 #include <stdio.h>
-#include <stdlib.h>
 #include <string.h>
 #include <zstd.h>
 
 #include "ruby.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/error.h"
 #include "zstds_ext/io.h"
 #include "zstds_ext/macro.h"
@@ -23,7 +20,7 @@ enum {
 
 // -- file --
 
-static inline zstds_ext_result_t read_file(FILE* source_file, uint8_t* source_buffer, size_t* source_length_ptr, size_t source_buffer_length)
+static inline zstds_ext_result_t read_file(FILE* source_file, zstds_ext_byte_t* source_buffer, size_t* source_length_ptr, size_t source_buffer_length)
 {
   size_t read_length = fread(source_buffer, 1, source_buffer_length, source_file);
   if (read_length == 0 && feof(source_file)) {
@@ -39,7 +36,7 @@ static inline zstds_ext_result_t read_file(FILE* source_file, uint8_t* source_bu
   return 0;
 }
 
-static inline zstds_ext_result_t write_file(FILE* destination_file, uint8_t* destination_buffer, size_t destination_length)
+static inline zstds_ext_result_t write_file(FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t destination_length)
 {
   size_t written_length = fwrite(destination_buffer, 1, destination_length, destination_file);
   if (written_length != destination_length) {
@@ -52,15 +49,15 @@ static inline zstds_ext_result_t write_file(FILE* destination_file, uint8_t* des
 // -- buffer --
 
 static inline zstds_ext_result_t create_buffers(
-  uint8_t** source_buffer_ptr, size_t source_buffer_length,
-  uint8_t** destination_buffer_ptr, size_t destination_buffer_length)
+  zstds_ext_byte_t** source_buffer_ptr, size_t source_buffer_length,
+  zstds_ext_byte_t** destination_buffer_ptr, size_t destination_buffer_length)
 {
-  uint8_t* source_buffer = malloc(source_buffer_length);
+  zstds_ext_byte_t* source_buffer = malloc(source_buffer_length);
   if (source_buffer == NULL) {
     return ZSTDS_EXT_ERROR_ALLOCATE_FAILED;
   }
 
-  uint8_t* destination_buffer = malloc(destination_buffer_length);
+  zstds_ext_byte_t* destination_buffer = malloc(destination_buffer_length);
   if (destination_buffer == NULL) {
     free(source_buffer);
     return ZSTDS_EXT_ERROR_ALLOCATE_FAILED;
@@ -79,12 +76,12 @@ static inline zstds_ext_result_t create_buffers(
 // Algorithm can use same buffer again.
 
 static inline zstds_ext_result_t read_more_source(
-  FILE*           source_file,
-  const uint8_t** source_ptr, size_t* source_length_ptr,
-  uint8_t* source_buffer, size_t source_buffer_length)
+  FILE*                    source_file,
+  const zstds_ext_byte_t** source_ptr, size_t* source_length_ptr,
+  zstds_ext_byte_t* source_buffer, size_t source_buffer_length)
 {
-  const uint8_t* source        = *source_ptr;
-  size_t         source_length = *source_length_ptr;
+  const zstds_ext_byte_t* source        = *source_ptr;
+  size_t                  source_length = *source_length_ptr;
 
   if (source != source_buffer) {
     if (source_length != 0) {
@@ -101,8 +98,8 @@ static inline zstds_ext_result_t read_more_source(
     return ZSTDS_EXT_ERROR_NOT_ENOUGH_SOURCE_BUFFER;
   }
 
-  uint8_t* remaining_source_buffer = source_buffer + source_length;
-  size_t   new_source_length;
+  zstds_ext_byte_t* remaining_source_buffer = source_buffer + source_length;
+  size_t            new_source_length;
 
   zstds_ext_result_t ext_result = read_file(source_file, remaining_source_buffer, &new_source_length, remaining_source_buffer_length);
   if (ext_result != 0) {
@@ -157,8 +154,8 @@ static inline zstds_ext_result_t read_more_source(
 // Than algorithm can use same buffer again.
 
 static inline zstds_ext_result_t flush_destination_buffer(
-  FILE*    destination_file,
-  uint8_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
+  FILE*             destination_file,
+  zstds_ext_byte_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
 {
   if (*destination_length_ptr == 0) {
     // We want to write more data at once, than buffer has.
@@ -175,7 +172,7 @@ static inline zstds_ext_result_t flush_destination_buffer(
   return 0;
 }
 
-static inline zstds_ext_result_t write_remaining_destination(FILE* destination_file, uint8_t* destination_buffer, size_t destination_length)
+static inline zstds_ext_result_t write_remaining_destination(FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t destination_length)
 {
   if (destination_length == 0) {
     return 0;
@@ -200,9 +197,9 @@ static inline zstds_ext_result_t write_remaining_destination(FILE* destination_f
 // -- compress --
 
 static inline zstds_ext_result_t buffered_compress(
-  ZSTD_CCtx*      ctx,
-  const uint8_t** source_ptr, size_t* source_length_ptr,
-  FILE* destination_file, uint8_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
+  ZSTD_CCtx*               ctx,
+  const zstds_ext_byte_t** source_ptr, size_t* source_length_ptr,
+  FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
 {
   zstds_result_t     result;
   zstds_ext_result_t ext_result;
@@ -249,7 +246,7 @@ static inline zstds_ext_result_t buffered_compress(
 
 static inline zstds_ext_result_t buffered_compressor_finish(
   ZSTD_CCtx* ctx,
-  FILE* destination_file, uint8_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
+  FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
 {
   zstds_result_t     result;
   zstds_ext_result_t ext_result;
@@ -293,14 +290,14 @@ static inline zstds_ext_result_t buffered_compressor_finish(
 
 static inline zstds_ext_result_t compress(
   ZSTD_CCtx* ctx,
-  FILE* source_file, uint8_t* source_buffer, size_t source_buffer_length,
-  FILE* destination_file, uint8_t* destination_buffer, size_t destination_buffer_length)
+  FILE* source_file, zstds_ext_byte_t* source_buffer, size_t source_buffer_length,
+  FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t destination_buffer_length)
 {
   zstds_ext_result_t ext_result;
 
-  const uint8_t* source             = source_buffer;
-  size_t         source_length      = 0;
-  size_t         destination_length = 0;
+  const zstds_ext_byte_t* source             = source_buffer;
+  size_t                  source_length      = 0;
+  size_t                  destination_length = 0;
 
   BUFFERED_READ_SOURCE(
     buffered_compress,
@@ -346,8 +343,8 @@ VALUE zstds_ext_compress_io(VALUE ZSTDS_EXT_UNUSED(self), VALUE source, VALUE de
     destination_buffer_length = ZSTD_CStreamOutSize();
   }
 
-  uint8_t* source_buffer;
-  uint8_t* destination_buffer;
+  zstds_ext_byte_t* source_buffer;
+  zstds_ext_byte_t* destination_buffer;
 
   ext_result = create_buffers(
     &source_buffer, source_buffer_length,
@@ -380,9 +377,9 @@ VALUE zstds_ext_compress_io(VALUE ZSTDS_EXT_UNUSED(self), VALUE source, VALUE de
 // -- decompress --
 
 static inline zstds_ext_result_t buffered_decompress(
-  ZSTD_DCtx*      ctx,
-  const uint8_t** source_ptr, size_t* source_length_ptr,
-  FILE* destination_file, uint8_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
+  ZSTD_DCtx*               ctx,
+  const zstds_ext_byte_t** source_ptr, size_t* source_length_ptr,
+  FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t* destination_length_ptr, size_t destination_buffer_length)
 {
   zstds_result_t     result;
   zstds_ext_result_t ext_result;
@@ -429,14 +426,14 @@ static inline zstds_ext_result_t buffered_decompress(
 
 static inline zstds_ext_result_t decompress(
   ZSTD_DCtx* ctx,
-  FILE* source_file, uint8_t* source_buffer, size_t source_buffer_length,
-  FILE* destination_file, uint8_t* destination_buffer, size_t destination_buffer_length)
+  FILE* source_file, zstds_ext_byte_t* source_buffer, size_t source_buffer_length,
+  FILE* destination_file, zstds_ext_byte_t* destination_buffer, size_t destination_buffer_length)
 {
   zstds_ext_result_t ext_result;
 
-  const uint8_t* source             = source_buffer;
-  size_t         source_length      = 0;
-  size_t         destination_length = 0;
+  const zstds_ext_byte_t* source             = source_buffer;
+  size_t                  source_length      = 0;
+  size_t                  destination_length = 0;
 
   BUFFERED_READ_SOURCE(
     buffered_decompress,
@@ -474,8 +471,8 @@ VALUE zstds_ext_decompress_io(VALUE ZSTDS_EXT_UNUSED(self), VALUE source, VALUE
     destination_buffer_length = ZSTD_DStreamOutSize();
   }
 
-  uint8_t* source_buffer;
-  uint8_t* destination_buffer;
+  zstds_ext_byte_t* source_buffer;
+  zstds_ext_byte_t* destination_buffer;
 
   ext_result = create_buffers(
     &source_buffer, source_buffer_length,

data/ext/zstds_ext/main.c

@@ -3,7 +3,6 @@
 
 #include "ruby.h"
 #include "zstds_ext/buffer.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/dictionary.h"
 #include "zstds_ext/io.h"
 #include "zstds_ext/option.h"

data/ext/zstds_ext/option.c

@@ -3,13 +3,9 @@
 
 #include "zstds_ext/option.h"
 
-#include <stdbool.h>
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/dictionary.h"
 #include "zstds_ext/error.h"
 

data/ext/zstds_ext/option.h

@@ -5,8 +5,6 @@
 #define ZSTDS_EXT_OPTIONS_H
 
 #include <stdbool.h>
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
@@ -22,9 +20,9 @@ enum {
   ZSTDS_EXT_OPTION_TYPE_STRATEGY
 };
 
-typedef uint_fast8_t       zstds_ext_option_type_t;
-typedef int                zstds_ext_option_value_t;
-typedef unsigned long long zstds_ext_ull_option_value_t;
+typedef zstds_ext_byte_fast_t zstds_ext_option_type_t;
+typedef int                   zstds_ext_option_value_t;
+typedef unsigned long long    zstds_ext_ull_option_value_t;
 
 typedef struct {
   bool                     has_value;

data/ext/zstds_ext/stream/compressor.c

@@ -3,12 +3,9 @@
 
 #include "zstds_ext/stream/compressor.h"
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/error.h"
 #include "zstds_ext/option.h"
 
@@ -19,7 +16,7 @@ static void free_compressor(zstds_ext_compressor_t* compressor_ptr)
     ZSTD_freeCCtx(ctx);
   }
 
-  uint8_t* destination_buffer = compressor_ptr->destination_buffer;
+  zstds_ext_byte_t* destination_buffer = compressor_ptr->destination_buffer;
   if (destination_buffer != NULL) {
     free(destination_buffer);
   }
@@ -68,7 +65,7 @@ VALUE zstds_ext_initialize_compressor(VALUE self, VALUE options)
     destination_buffer_length = ZSTD_CStreamOutSize();
   }
 
-  uint8_t* destination_buffer = malloc(destination_buffer_length);
+  zstds_ext_byte_t* destination_buffer = malloc(destination_buffer_length);
   if (destination_buffer == NULL) {
     ZSTD_freeCCtx(ctx);
     zstds_ext_raise_error(ZSTDS_EXT_ERROR_ALLOCATE_FAILED);
@@ -185,9 +182,9 @@ VALUE zstds_ext_compressor_read_result(VALUE self)
   GET_COMPRESSOR(self);
   DO_NOT_USE_AFTER_CLOSE(compressor_ptr);
 
-  uint8_t* destination_buffer                  = compressor_ptr->destination_buffer;
-  size_t   destination_buffer_length           = compressor_ptr->destination_buffer_length;
-  size_t   remaining_destination_buffer_length = compressor_ptr->remaining_destination_buffer_length;
+  zstds_ext_byte_t* destination_buffer                  = compressor_ptr->destination_buffer;
+  size_t            destination_buffer_length           = compressor_ptr->destination_buffer_length;
+  size_t            remaining_destination_buffer_length = compressor_ptr->remaining_destination_buffer_length;
 
   const char* result        = (const char*)destination_buffer;
   size_t      result_length = destination_buffer_length - remaining_destination_buffer_length;
@@ -212,7 +209,7 @@ VALUE zstds_ext_compressor_close(VALUE self)
     compressor_ptr->ctx = NULL;
   }
 
-  uint8_t* destination_buffer = compressor_ptr->destination_buffer;
+  zstds_ext_byte_t* destination_buffer = compressor_ptr->destination_buffer;
   if (destination_buffer != NULL) {
     free(destination_buffer);
 

data/ext/zstds_ext/stream/compressor.h

@@ -4,18 +4,17 @@
 #if !defined(ZSTDS_EXT_STREAM_COMPRESSOR_H)
 #define ZSTDS_EXT_STREAM_COMPRESSOR_H
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
+#include "zstds_ext/common.h"
 
 typedef struct {
-  ZSTD_CCtx* ctx;
-  uint8_t*   destination_buffer;
-  size_t     destination_buffer_length;
-  uint8_t*   remaining_destination_buffer;
-  size_t     remaining_destination_buffer_length;
+  ZSTD_CCtx*        ctx;
+  zstds_ext_byte_t* destination_buffer;
+  size_t            destination_buffer_length;
+  zstds_ext_byte_t* remaining_destination_buffer;
+  size_t            remaining_destination_buffer_length;
 } zstds_ext_compressor_t;
 
 VALUE zstds_ext_allocate_compressor(VALUE klass);

data/ext/zstds_ext/stream/decompressor.c

@@ -3,12 +3,9 @@
 
 #include "zstds_ext/stream/decompressor.h"
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/error.h"
 #include "zstds_ext/option.h"
 
@@ -19,7 +16,7 @@ static void free_decompressor(zstds_ext_decompressor_t* decompressor_ptr)
     ZSTD_freeDCtx(ctx);
   }
 
-  uint8_t* destination_buffer = decompressor_ptr->destination_buffer;
+  zstds_ext_byte_t* destination_buffer = decompressor_ptr->destination_buffer;
   if (destination_buffer != NULL) {
     free(destination_buffer);
   }
@@ -68,7 +65,7 @@ VALUE zstds_ext_initialize_decompressor(VALUE self, VALUE options)
     destination_buffer_length = ZSTD_DStreamOutSize();
   }
 
-  uint8_t* destination_buffer = malloc(destination_buffer_length);
+  zstds_ext_byte_t* destination_buffer = malloc(destination_buffer_length);
   if (destination_buffer == NULL) {
     ZSTD_freeDCtx(ctx);
     zstds_ext_raise_error(ZSTDS_EXT_ERROR_ALLOCATE_FAILED);
@@ -129,9 +126,9 @@ VALUE zstds_ext_decompressor_read_result(VALUE self)
   GET_DECOMPRESSOR(self);
   DO_NOT_USE_AFTER_CLOSE(decompressor_ptr);
 
-  uint8_t* destination_buffer                  = decompressor_ptr->destination_buffer;
-  size_t   destination_buffer_length           = decompressor_ptr->destination_buffer_length;
-  size_t   remaining_destination_buffer_length = decompressor_ptr->remaining_destination_buffer_length;
+  zstds_ext_byte_t* destination_buffer                  = decompressor_ptr->destination_buffer;
+  size_t            destination_buffer_length           = decompressor_ptr->destination_buffer_length;
+  size_t            remaining_destination_buffer_length = decompressor_ptr->remaining_destination_buffer_length;
 
   const char* result        = (const char*)destination_buffer;
   size_t      result_length = destination_buffer_length - remaining_destination_buffer_length;
@@ -156,7 +153,7 @@ VALUE zstds_ext_decompressor_close(VALUE self)
     decompressor_ptr->ctx = NULL;
   }
 
-  uint8_t* destination_buffer = decompressor_ptr->destination_buffer;
+  zstds_ext_byte_t* destination_buffer = decompressor_ptr->destination_buffer;
   if (destination_buffer != NULL) {
     free(destination_buffer);
 

data/ext/zstds_ext/stream/decompressor.h

@@ -4,18 +4,17 @@
 #if !defined(ZSTDS_EXT_STREAM_DECOMPRESSOR_H)
 #define ZSTDS_EXT_STREAM_DECOMPRESSOR_H
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
+#include "zstds_ext/common.h"
 
 typedef struct {
-  ZSTD_DCtx* ctx;
-  uint8_t*   destination_buffer;
-  size_t     destination_buffer_length;
-  uint8_t*   remaining_destination_buffer;
-  size_t     remaining_destination_buffer_length;
+  ZSTD_DCtx*        ctx;
+  zstds_ext_byte_t* destination_buffer;
+  size_t            destination_buffer_length;
+  zstds_ext_byte_t* remaining_destination_buffer;
+  size_t            remaining_destination_buffer_length;
 } zstds_ext_decompressor_t;
 
 VALUE zstds_ext_allocate_decompressor(VALUE klass);

data/ext/zstds_ext/string.c

@@ -3,13 +3,10 @@
 
 #include "zstds_ext/string.h"
 
-#include <stdint.h>
-#include <stdlib.h>
 #include <zstd.h>
 
 #include "ruby.h"
 #include "zstds_ext/buffer.h"
-#include "zstds_ext/common.h"
 #include "zstds_ext/error.h"
 #include "zstds_ext/macro.h"
 #include "zstds_ext/option.h"
@@ -65,7 +62,7 @@ static inline zstds_ext_result_t compress(
   size_t         remaining_destination_buffer_length = destination_buffer_length;
 
   while (true) {
-    out_buffer.dst  = (uint8_t*)RSTRING_PTR(destination_value) + destination_length;
+    out_buffer.dst  = (zstds_ext_byte_t*)RSTRING_PTR(destination_value) + destination_length;
     out_buffer.size = remaining_destination_buffer_length;
     out_buffer.pos  = 0;
 
@@ -166,7 +163,7 @@ static inline zstds_ext_result_t decompress(
   size_t         remaining_destination_buffer_length = destination_buffer_length;
 
   while (true) {
-    out_buffer.dst  = (uint8_t*)RSTRING_PTR(destination_value) + destination_length;
+    out_buffer.dst  = (zstds_ext_byte_t*)RSTRING_PTR(destination_value) + destination_length;
     out_buffer.size = remaining_destination_buffer_length;
     out_buffer.pos  = 0;
 

data/lib/zstds/stream/abstract.rb

@@ -23,17 +23,18 @@ module ZSTDS
       attr_reader :internal_encoding
       attr_reader :transcode_options
       attr_reader :pos
+
       alias tell pos
 
-      def initialize(io, external_encoding: nil, internal_encoding: nil, transcode_options: {})
+      def initialize(io, options = {})
         @raw_stream = create_raw_stream
 
         Validation.validate_io io
         @io = io
 
-        @stat = Stat.new @io.stat
+        @stat = Stat.new @io.stat if @io.respond_to? :stat
 
-        set_encoding external_encoding, internal_encoding, transcode_options
+        set_encoding options[:external_encoding], options[:internal_encoding], options[:transcode_options]
         reset_buffer
         reset_io_advise
 
@@ -50,8 +51,8 @@ module ZSTDS
 
       protected def reset_io_advise
         # Both compressor and decompressor need sequential io access.
-        @io.advise :sequential
-      rescue ::Errno::ESPIPE # rubocop:disable Lint/SuppressedException
+        @io.advise :sequential if @io.respond_to? :advise
+      rescue ::Errno::ESPIPE
         # ok
       end
 
@@ -126,7 +127,8 @@ module ZSTDS
       def rewind
         @raw_stream = create_raw_stream
 
-        @io.rewind
+        @io.rewind if @io.respond_to? :rewind
+
         reset_buffer
         reset_io_advise
 

data/lib/zstds/stream/reader.rb

@@ -147,7 +147,7 @@ module ZSTDS
       end
 
       protected def transcode_to_internal(data)
-        data = data.encode @internal_encoding, @transcode_options unless @internal_encoding.nil?
+        data = data.encode @internal_encoding, **@transcode_options unless @internal_encoding.nil?
         data
       end
 

data/lib/zstds/stream/writer.rb

@@ -137,7 +137,7 @@ module ZSTDS
       # -- common --
 
       protected def transcode(data)
-        data = data.encode @external_encoding, @transcode_options unless @external_encoding.nil?
+        data = data.encode @external_encoding, **@transcode_options unless @external_encoding.nil?
         data
       end
     end

data/lib/zstds/stream/writer_helpers.rb

@@ -13,13 +13,13 @@ module ZSTDS
         write object
       end
 
-      def print(*objects)
+      def print(*objects, field_separator: $OUTPUT_FIELD_SEPARATOR, record_separator: $OUTPUT_RECORD_SEPARATOR)
         objects.each do |object|
           write object
-          write $OUTPUT_FIELD_SEPARATOR unless $OUTPUT_FIELD_SEPARATOR.nil?
+          write field_separator unless field_separator.nil?
         end
 
-        write $OUTPUT_RECORD_SEPARATOR unless $OUTPUT_RECORD_SEPARATOR.nil?
+        write record_separator unless record_separator.nil?
 
         nil
       end

data/lib/zstds/validation.rb

@@ -5,6 +5,19 @@ require_relative "error"
 
 module ZSTDS
   module Validation
+    IO_METHODS = %i[
+      read
+      write
+      readpartial
+      read_nonblock
+      write_nonblock
+      eof?
+      flush
+      close
+      closed?
+    ]
+    .freeze
+
     def self.validate_bool(value)
       raise ValidateError, "invalid bool" unless value.is_a?(::TrueClass) || value.is_a?(::FalseClass)
     end
@@ -30,7 +43,7 @@ module ZSTDS
     end
 
     def self.validate_io(value)
-      raise ValidateError, "invalid io" unless value.is_a? ::IO
+      raise ValidateError, "invalid io" unless IO_METHODS.all? { |method| value.respond_to? method }
     end
 
     def self.validate_array(value)

data/lib/zstds/version.rb

@@ -2,5 +2,5 @@
 # Copyright (c) 2019 AUTHORS, MIT License.
 
 module ZSTDS
-  VERSION = "1.0.2".freeze
+  VERSION = "1.0.3".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-zstds
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Andrew Aladjev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-02 00:00:00.000000000 Z
+date: 2020-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: codecov
@@ -217,7 +217,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Ruby bindings for zstd library.