]> gitweb.hhaalo.de Git - sane-kds-s2000w-net.git/commitdiff
remove read size use instead read info
authorBastian Dehn <hhaalo@arcor.de>
Sun, 11 Feb 2024 19:19:42 +0000 (20:19 +0100)
committerBastian Dehn <hhaalo@arcor.de>
Sun, 11 Feb 2024 19:20:07 +0000 (20:20 +0100)
src/kds_s2000w_net.c
tests/kds_s2000w_net_tests.c

index 97ad293707d3c19627485d3bc64a222cedd7e953..f88b38dbdd4d1a31d1bc865cd658db3e7f145ddd 100644 (file)
@@ -8,11 +8,11 @@
 
 typedef struct {
        int cancel;
-       metadata current_metadata;
        int read_size;
        int scan_started;
        int readed_bytes_per_line;
        int readed_lines;
+       metadata current_metadata;
 } readinfo;
 
 readinfo read_info;
@@ -24,13 +24,9 @@ readinfo* get_read_info()
        return &read_info;
 }
 
-[[deprecated("use reade_info instead")]]
-int cancel = 0;
 [[deprecated("use reade_info instead")]]
 metadata current_metadata;
 [[deprecated("use reade_info instead")]]
-int read_size = 0;
-[[deprecated("use reade_info instead")]]
 int scan_started = 0;
 [[deprecated("use reade_info instead")]]
 int readed_bytes_per_line = 0;
@@ -43,18 +39,6 @@ metadata* get_current_metadata()
        return &current_metadata;
 }
 
-[[deprecated("use reade_info instead")]]
-int* get_read_size()
-{
-       return &read_size;
-}
-
-[[deprecated("use reade_info instead")]]
-int* get_cancel()
-{
-       return &cancel;
-}
-
 [[deprecated("use reade_info instead")]]
 int* get_readed_lines()
 {
@@ -273,7 +257,7 @@ SANE_Status _sane_kds_s2000w_net_get_parameters(SANE_Handle handle,
        params->pixels_per_line = current_metadata.pixels_per_line;
        params->lines = current_metadata.lines;
        params->depth = current_metadata.depth;
-       read_size = 0;
+       read_info.read_size = 0;
        readed_lines = 0;
        readed_bytes_per_line = 0;
 
@@ -306,8 +290,8 @@ SANE_Status _sane_kds_s2000w_net_read(SANE_Handle handle, SANE_Byte* data,
        }
 
        int skip_header_bytes = _sane_kds_s2000w_net_find_first_data_byte(current_metadata.image);
-       if (read_size <= 0)
-               read_size += skip_header_bytes;
+       if (read_info.read_size <= 0)
+               read_info.read_size += skip_header_bytes;
 
        if (readed_lines >= current_metadata.lines) {
                *length = 0;
@@ -320,10 +304,10 @@ SANE_Status _sane_kds_s2000w_net_read(SANE_Handle handle, SANE_Byte* data,
                maxlen = current_metadata.bytes_per_line - readed_bytes_per_line;
 
        *length = maxlen;
-       memcpy(data, current_metadata.image + read_size, *length);
+       memcpy(data, current_metadata.image + read_info.read_size, *length);
 
        readed_bytes_per_line += *length;
-       read_size += *length;
+       read_info.read_size += *length;
 
        if (readed_bytes_per_line >= current_metadata.bytes_per_line) {
                readed_bytes_per_line = 0;
index 2311f03d6736de686ac62a43cc1c8bc9ffc8f9ef..d62977c777bfd5955fdccc8c63ebcf52715efab5 100644 (file)
@@ -49,8 +49,7 @@ START_TEST(sane_kds_s2000w_net_read_all_lines_test)
        readinfo* read_info = get_read_info();
        read_info->cancel = 0;
        metadata* current_metadata_ptr = get_current_metadata();
-       int* read_size = get_read_size();
-       *read_size = 0;
+       read_info->read_size = 0;
        char* image = malloc(sizeof(char) * 53);
        for (int i = 0; i < 3; i++) {
                image[i] = 0x0a;
@@ -74,7 +73,7 @@ START_TEST(sane_kds_s2000w_net_read_all_lines_test)
        }
 
        ck_assert_int_eq(status, SANE_STATUS_EOF);
-       ck_assert_int_eq(*read_size, 53);
+       ck_assert_int_eq(read_info->read_size, 53);
 
        free(dataptr);
        dataptr = NULL;
@@ -87,11 +86,9 @@ END_TEST
 
 START_TEST(sane_kds_s2000w_net_read_test)
 {
-       int* read_size_ptr = get_read_size();
-       *read_size_ptr = 0;
+       readinfo* read_info = get_read_info();
        metadata* current_metadata_ptr = get_current_metadata();
-       int* read_size = get_read_size();
-       *read_size = 0;
+       read_info->read_size = 0;
        current_metadata_ptr->lines = 1;
        current_metadata_ptr->bytes_per_line = 65536;
        char* image = malloc(sizeof(char) * 65539);
@@ -109,7 +106,7 @@ START_TEST(sane_kds_s2000w_net_read_test)
 
        ck_assert_int_eq(*length, 65536);
        ck_assert_mem_eq(dataptr, image + 3, 65536);
-       ck_assert_int_eq(*read_size_ptr, 65539);
+       ck_assert_int_eq(read_info->read_size, 65539);
 
        free(dataptr);
        dataptr = NULL;
@@ -123,11 +120,9 @@ END_TEST
 
 START_TEST(sane_kds_s2000w_net_read_bytes_per_line_bigger_test)
 {
-       int* read_size_ptr = get_read_size();
-       *read_size_ptr = 0;
+       readinfo* read_info = get_read_info();
+       read_info->read_size = 0;
        metadata* current_metadata_ptr = get_current_metadata();
-       int* read_size = get_read_size();
-       *read_size = 0;
        int* readed_bytes_per_line = get_readed_bytes_per_line();
        readed_bytes_per_line = 0;
        current_metadata_ptr->lines = 1;
@@ -146,7 +141,7 @@ START_TEST(sane_kds_s2000w_net_read_bytes_per_line_bigger_test)
        _sane_kds_s2000w_net_read(NULL, (void*) dataptr, maxlen, length);
        _sane_kds_s2000w_net_read(NULL, (void*) dataptr, maxlen, length);
 
-       ck_assert_int_eq(*read_size_ptr, 95003);        
+       ck_assert_int_eq(read_info->read_size, 95003);
 
        free(dataptr);
        dataptr = NULL;