Skip to content

Commit bcf0447

Browse files
committed
Make helper functions void
As reported by #65
1 parent e032c79 commit bcf0447

File tree

1 file changed

+20
-40
lines changed

1 file changed

+20
-40
lines changed

src/read_data_impl.h

+20-40
Original file line numberDiff line numberDiff line change
@@ -336,12 +336,11 @@ READ_TYPED_FUNC1(mat_t *mat, READ_TYPE *data, enum matio_types data_type, size_t
336336
#define READ_TYPE_UINT64_DATA CAT(READ_TYPED_FUNC2, UInt64)
337337
#endif /* HAVE_MAT_UINT64_T */
338338

339-
static size_t
339+
static void
340340
READ_TYPE_DOUBLE_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
341341
{
342-
size_t bytesread = 0;
343342
#if READ_TYPE_TYPE == READ_TYPE_DOUBLE
344-
bytesread = InflateData(mat, z, data, len*sizeof(double));
343+
InflateData(mat, z, data, len*sizeof(double));
345344
if ( mat->byteswap ) {
346345
mat_uint32_t i;
347346
for ( i = 0; i < len; i++ ) {
@@ -354,15 +353,13 @@ READ_TYPE_DOUBLE_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len
354353
double v[READ_BLOCK_SIZE/sizeof(double)];
355354
READ_COMPRESSED_DATA(READ_TYPE, Mat_doubleSwap);
356355
#endif
357-
return bytesread;
358356
}
359357

360-
static size_t
358+
static void
361359
READ_TYPE_SINGLE_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
362360
{
363-
size_t bytesread = 0;
364361
#if READ_TYPE_TYPE == READ_TYPE_SINGLE
365-
bytesread = InflateData(mat, z, data, len*sizeof(float));
362+
InflateData(mat, z, data, len*sizeof(float));
366363
if ( mat->byteswap ) {
367364
mat_uint32_t i;
368365
for ( i = 0; i < len; i++ ) {
@@ -375,16 +372,14 @@ READ_TYPE_SINGLE_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len
375372
float v[READ_BLOCK_SIZE/sizeof(float)];
376373
READ_COMPRESSED_DATA(READ_TYPE, Mat_floatSwap);
377374
#endif
378-
return bytesread;
379375
}
380376

381377
#ifdef HAVE_MAT_INT64_T
382-
static size_t
378+
static void
383379
READ_TYPE_INT64_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
384380
{
385-
size_t bytesread = 0;
386381
#if READ_TYPE_TYPE == READ_TYPE_INT64
387-
bytesread = InflateData(mat, z, data, len*sizeof(mat_int64_t));
382+
InflateData(mat, z, data, len*sizeof(mat_int64_t));
388383
if ( mat->byteswap ) {
389384
mat_uint32_t i;
390385
for ( i = 0; i < len; i++ ) {
@@ -397,17 +392,15 @@ READ_TYPE_INT64_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
397392
mat_int64_t v[READ_BLOCK_SIZE/sizeof(mat_int64_t)];
398393
READ_COMPRESSED_DATA(READ_TYPE, Mat_int64Swap);
399394
#endif
400-
return bytesread;
401395
}
402396
#endif /* HAVE_MAT_INT64_T */
403397

404398
#ifdef HAVE_MAT_UINT64_T
405-
static size_t
399+
static void
406400
READ_TYPE_UINT64_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
407401
{
408-
size_t bytesread = 0;
409402
#if READ_TYPE_TYPE == READ_TYPE_UINT64
410-
bytesread = InflateData(mat, z, data, len*sizeof(mat_uint64_t));
403+
InflateData(mat, z, data, len*sizeof(mat_uint64_t));
411404
if ( mat->byteswap ) {
412405
mat_uint32_t i;
413406
for ( i = 0; i < len; i++ ) {
@@ -420,16 +413,14 @@ READ_TYPE_UINT64_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len
420413
mat_uint64_t v[READ_BLOCK_SIZE/sizeof(mat_uint64_t)];
421414
READ_COMPRESSED_DATA(READ_TYPE, Mat_uint64Swap);
422415
#endif
423-
return bytesread;
424416
}
425417
#endif /* HAVE_MAT_UINT64_T */
426418

427-
static size_t
419+
static void
428420
READ_TYPE_INT32_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
429421
{
430-
size_t bytesread = 0;
431422
#if READ_TYPE_TYPE == READ_TYPE_INT32
432-
bytesread = InflateData(mat, z, data, len*sizeof(mat_int32_t));
423+
InflateData(mat, z, data, len*sizeof(mat_int32_t));
433424
if ( mat->byteswap ) {
434425
mat_uint32_t i;
435426
for ( i = 0; i < len; i++ ) {
@@ -442,15 +433,13 @@ READ_TYPE_INT32_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
442433
mat_int32_t v[READ_BLOCK_SIZE/sizeof(mat_int32_t)];
443434
READ_COMPRESSED_DATA(READ_TYPE, Mat_int32Swap);
444435
#endif
445-
return bytesread;
446436
}
447437

448-
static size_t
438+
static void
449439
READ_TYPE_UINT32_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
450440
{
451-
size_t bytesread = 0;
452441
#if READ_TYPE_TYPE == READ_TYPE_UINT32
453-
bytesread = InflateData(mat, z, data, len*sizeof(mat_uint32_t));
442+
InflateData(mat, z, data, len*sizeof(mat_uint32_t));
454443
if ( mat->byteswap ) {
455444
mat_uint32_t i;
456445
for ( i = 0; i < len; i++ ) {
@@ -463,15 +452,13 @@ READ_TYPE_UINT32_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len
463452
mat_uint32_t v[READ_BLOCK_SIZE/sizeof(mat_uint32_t)];
464453
READ_COMPRESSED_DATA(READ_TYPE, Mat_uint32Swap);
465454
#endif
466-
return bytesread;
467455
}
468456

469-
static size_t
457+
static void
470458
READ_TYPE_INT16_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
471459
{
472-
size_t bytesread = 0;
473460
#if READ_TYPE_TYPE == READ_TYPE_INT16
474-
bytesread = InflateData(mat, z, data, len*sizeof(mat_int16_t));
461+
InflateData(mat, z, data, len*sizeof(mat_int16_t));
475462
if ( mat->byteswap ) {
476463
mat_uint32_t i;
477464
for ( i = 0; i < len; i++ ) {
@@ -484,15 +471,13 @@ READ_TYPE_INT16_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
484471
mat_int16_t v[READ_BLOCK_SIZE/sizeof(mat_int16_t)];
485472
READ_COMPRESSED_DATA(READ_TYPE, Mat_int16Swap);
486473
#endif
487-
return bytesread;
488474
}
489475

490-
static size_t
476+
static void
491477
READ_TYPE_UINT16_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
492478
{
493-
size_t bytesread = 0;
494479
#if READ_TYPE_TYPE == READ_TYPE_UINT16
495-
bytesread = InflateData(mat, z, data, len*sizeof(mat_uint16_t));
480+
InflateData(mat, z, data, len*sizeof(mat_uint16_t));
496481
if ( mat->byteswap ) {
497482
mat_uint32_t i;
498483
for ( i = 0; i < len; i++ ) {
@@ -505,37 +490,32 @@ READ_TYPE_UINT16_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len
505490
mat_uint16_t v[READ_BLOCK_SIZE/sizeof(mat_uint16_t)];
506491
READ_COMPRESSED_DATA(READ_TYPE, Mat_uint16Swap);
507492
#endif
508-
return bytesread;
509493
}
510494

511-
static size_t
495+
static void
512496
READ_TYPE_INT8_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
513497
{
514-
size_t bytesread = 0;
515498
#if READ_TYPE_TYPE == READ_TYPE_INT8
516-
bytesread = InflateData(mat, z, data, len*sizeof(mat_int8_t));
499+
InflateData(mat, z, data, len*sizeof(mat_int8_t));
517500
#else
518501
mat_uint32_t i;
519502
const size_t data_size = sizeof(mat_int8_t);
520503
mat_int8_t v[READ_BLOCK_SIZE/sizeof(mat_int8_t)];
521504
READ_COMPRESSED_DATA_NOSWAP(READ_TYPE);
522505
#endif
523-
return bytesread;
524506
}
525507

526-
static size_t
508+
static void
527509
READ_TYPE_UINT8_DATA(mat_t *mat, z_streamp z, READ_TYPE *data, mat_uint32_t len)
528510
{
529-
size_t bytesread = 0;
530511
#if READ_TYPE_TYPE == READ_TYPE_UINT8
531-
bytesread = InflateData(mat, z, data, len*sizeof(mat_uint8_t));
512+
InflateData(mat, z, data, len*sizeof(mat_uint8_t));
532513
#else
533514
mat_uint32_t i;
534515
const size_t data_size = sizeof(mat_uint8_t);
535516
mat_uint8_t v[READ_BLOCK_SIZE/sizeof(mat_uint8_t)];
536517
READ_COMPRESSED_DATA_NOSWAP(READ_TYPE);
537518
#endif
538-
return bytesread;
539519
}
540520

541521
/** @brief Reads data of type @c data_type into a READ_TYPE type

0 commit comments

Comments
 (0)