Splitting my_close() and return() into two lines.

Some functions such as my_read() and my_lseek()
use the return code -1 to indicate an error which
is an invalid exit code for the main program. For
these functions the my_close() and return() calls
are on seperate lines instead of combining them
into one operation.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/bindump.c b/src/bindump.c
index c9056a0..624ceb1 100644
--- a/src/bindump.c
+++ b/src/bindump.c
@@ -385,7 +385,8 @@ int my_lseek(BinDumpFile * file, size_t offset, uint32_t verbose)
    if ((lseek(file->fd, (off_t)offset, SEEK_SET) == -1))
    {
       perror(PROGRAM_NAME ": lseek()");
-      return(my_close(file, verbose));
+      my_close(file, verbose);
+      return(-1);
    };
    file->pos += offset;
    return(0);
@@ -480,7 +481,8 @@ int my_read(BinDumpFile * file, size_t offset, size_t len, uint32_t verbose)
    if ((file->code = read(file->fd, file->data, max)) == -1)
    {
       perror(PROGRAM_NAME ": read()");
-      return(my_close(file, verbose));
+      my_close(file, verbose);
+      return(-1);
    };
 
    if (file->code < 1)