]> Kevux Git Server - fll/commit
Update: Fix print function bugs and add missing functions.
authorKevin Day <thekevinday@gmail.com>
Tue, 9 Nov 2021 02:26:17 +0000 (20:26 -0600)
committerKevin Day <thekevinday@gmail.com>
Tue, 9 Nov 2021 02:26:17 +0000 (20:26 -0600)
commit6fb1fa00176bc43d2fb413e77357bddf006a9154
tree8b80ed0ba15b6e614ae3921e17f673d7ab8f8e1e
parent3b6d88de300df33200f5d96b1d5fc268db8ec7f6
Update: Fix print function bugs and add missing functions.

There are some cases where the logic is flawed and fails to print correctly.
Given a bash binary, passing it through the print functions and back to the disk in raw format resulted in a different binary.
The binary should be identical to the original and is not.

It seems that when I attempted to reduce the number of variables in the print function I failed to account for certain details.
This essentially reverts some of the previous logic and puts back the variables.

Many off these functions are almost identical but their differences result in requiring slightly different code.
This makes the code less manageable and easy to get lost in.
The code has been tweaked somewhat to be more consistent across the board, where possible.
This might come at a cost of some slight performance.

To make the code easier to read, separate the print functions into two groups:
1) The print functions that use file streams.
2) The print functions that use file descriptors.

This code is slightly different and while I could use macros to make this simpler, I opted to avoid macros at the cost of duplication.
Move these two groups into separate files, significantly improving code readability.

There is a mistake in the previous code where errno is being processed for file stream errors.
The libc file stream functions do not utilize errno.
Change the affected code to just return F_output with the error bit set.
I am unable to find a discrete list of error codes returned by ferror().
I will need to additional work to get these codes so I can properly map them to more verbose error codes.

The documentation is no longer out of sync.

There should be a file descriptor function equivalent for nearly every file stream function.
In this regard, several missing functions are now added.

This is an unplanned change that is rather large.
I am concerned with oversights and regressions.
Just keep an eye out for regressions and hope that there are none.
build/level_0/settings
build/monolithic/settings
level_0/f_print/c/print.c
level_0/f_print/c/print.h
level_0/f_print/c/print_to.c [new file with mode: 0644]
level_0/f_print/c/print_to.h [new file with mode: 0644]
level_0/f_print/c/private-print.c
level_0/f_print/c/private-print.h
level_0/f_print/c/private-print_to.c [new file with mode: 0644]
level_0/f_print/c/private-print_to.h [new file with mode: 0644]
level_0/f_print/data/build/settings