]> Kevux Git Server - fll/log
fll
2 years agoUpdate: Add unit tests for f_signal project.
Kevin Day [Wed, 8 Jun 2022 04:46:30 +0000 (23:46 -0500)]
Update: Add unit tests for f_signal project.

2 years agoUpdate: Rename f_thread_signal() to f_thread_signal_write().
Kevin Day [Wed, 8 Jun 2022 04:44:27 +0000 (23:44 -0500)]
Update: Rename f_thread_signal() to f_thread_signal_write().

I feel this function name should be more explicit.
That is, it should avoid confusion between read and write operations.

2 years agoBugfix: Problems exposed by unit tests for f_signal.
Kevin Day [Wed, 8 Jun 2022 04:38:07 +0000 (23:38 -0500)]
Bugfix: Problems exposed by unit tests for f_signal.

Fix ordering of functions.
Compare using "== -1" rather than "< 0" because the POSIX standard says "-1" rather than "less than zero".

The f_signal_read() function has several problems:
- Should be passing a pointer rather than an array to poll().
- Needs to check data_poll.revents.
- Missing the parameter checking.

2 years agoUpdate: Unit tests in several projects.
Kevin Day [Wed, 8 Jun 2022 01:13:59 +0000 (20:13 -0500)]
Update: Unit tests in several projects.

Remove invalid errnor checks.
Design some of the mocks to be more consistent across the projects.

2 years agoBugfix: Problems exposed by unit tests in f_file.
Kevin Day [Wed, 8 Jun 2022 01:11:45 +0000 (20:11 -0500)]
Bugfix: Problems exposed by unit tests in f_file.

The stream functions do not populate errno.
In these cases return either F_file_read or F_file_write with the error bits set as appropriate.

Update the documentation comments.

2 years agoCleanup: Update unit tests in f_print to follow current __wrap_fwrite_unlocked()...
Kevin Day [Tue, 7 Jun 2022 04:59:26 +0000 (23:59 -0500)]
Cleanup: Update unit tests in f_print to follow current __wrap_fwrite_unlocked() mock practices.

2 years agoCleanup: The f_print project code syntax, adding an inline function.
Kevin Day [Tue, 7 Jun 2022 04:58:24 +0000 (23:58 -0500)]
Cleanup: The f_print project code syntax, adding an inline function.

2 years agoUpdate: Get all f_conversion unit tests working.
Kevin Day [Tue, 7 Jun 2022 04:01:41 +0000 (23:01 -0500)]
Update: Get all f_conversion unit tests working.

Add missing mocks.

2 years agoBugfix: Problems exposed by unit tests in f_conversion.
Kevin Day [Tue, 7 Jun 2022 03:39:30 +0000 (22:39 -0500)]
Bugfix: Problems exposed by unit tests in f_conversion.

The unit tests were disabled due to previous problems that are not no longer a problem.
This exposed a few areas that could be improved.

Rename "output" to "stream".
Make sure to call ferror_unlocked() to test for error.
Add loops to handle when fwrite_unlocked() doesn't write all the data but also doesn't throw an error.

Use "negative_or_zero" instead of "negative" to be more verbose and less confusing.

2 years agoBugfix: When any of -c, -p, or -s is the last parameter in test.sh no error is thrown.
Kevin Day [Mon, 6 Jun 2022 00:21:48 +0000 (19:21 -0500)]
Bugfix: When any of -c, -p, or -s is the last parameter in test.sh no error is thrown.

These must have an error because they expect an additional argument.
Change the behavior of the code to cause an error in this cases.

I took the lazy way out and just set the variables to trigger an error rather than implementing a specific error handler for these cases.

2 years agoUpdate: Next micro version (0.5.10).
Kevin Day [Mon, 6 Jun 2022 00:19:51 +0000 (19:19 -0500)]
Update: Next micro version (0.5.10).

2 years agoCleanup: Fully remove stale/unused variables.
Kevin Day [Sun, 5 Jun 2022 22:57:27 +0000 (17:57 -0500)]
Cleanup: Fully remove stale/unused variables.

I simply assumed the compiler warnings meant that the variables were not used at all.
Looks like this is an incorrect assumption.
The compilers appear to more intelligent than I expected and can detect that even though the variables are modified later on, they still are not used.

This is good news.
The bad news is that means I have bad code resulting from an incomplete clean up that need to be fixed.
This bad code is now fixed.

2 years agoBugfix: Package building test failures do not properly fail.
Kevin Day [Sun, 5 Jun 2022 22:48:04 +0000 (17:48 -0500)]
Bugfix: Package building test failures do not properly fail.

The wrong variable is checked and so the failure is missed.

2 years agoUpdate: Add unit tests for f_status.
Kevin Day [Sun, 5 Jun 2022 22:42:57 +0000 (17:42 -0500)]
Update: Add unit tests for f_status.

The f_status only has macros and I generally do not test macros.
These are special macros used everywhere so I decided to add macro unit tests.

2 years agoBugfix: Fix problems exposed by unit tests in f_status.
Kevin Day [Sun, 5 Jun 2022 22:40:11 +0000 (17:40 -0500)]
Bugfix: Fix problems exposed by unit tests in f_status.

The signal bit can be confused with the error and warning bits.
The tests for checking if the signal bit is set need to test exclusively for the signal bit being set.
The tests for the error and warning bits do not care if this is a signal.
This is done to simplify some of the logic.
In the cases where the signal bit is to be tested, first check the signal bit and then check for the othr bits.

2 years agoCleanup: Remove stale/unused variables.
Kevin Day [Sun, 5 Jun 2022 21:08:20 +0000 (16:08 -0500)]
Cleanup: Remove stale/unused variables.

2 years agoUpdate: Improve the logic in the string comparison tests.
Kevin Day [Sun, 5 Jun 2022 21:03:14 +0000 (16:03 -0500)]
Update: Improve the logic in the string comparison tests.

Several of the F_parameter errors do not need to be returned.
This logic is already handled by the called functions as not matched.

Make sure the stop range is not longer than the buffer used.
When the buffer used is smaller than the requested range stop, then use the buffer used rather than the stop point.

2 years agoCleanup: Syntax.
Kevin Day [Sun, 5 Jun 2022 21:02:42 +0000 (16:02 -0500)]
Cleanup: Syntax.

2 years agoBugfix: Incorrect variable used results in F_parameter error.
Kevin Day [Sun, 5 Jun 2022 21:02:25 +0000 (16:02 -0500)]
Bugfix: Incorrect variable used results in F_parameter error.

2 years agoUpdate: Implement Math Symbol Category.
Kevin Day [Sun, 5 Jun 2022 17:22:33 +0000 (12:22 -0500)]
Update: Implement Math Symbol Category.

2 years agoUpdate: Handle more cases of combining characters following white space.
Kevin Day [Sat, 4 Jun 2022 03:40:29 +0000 (22:40 -0500)]
Update: Handle more cases of combining characters following white space.

The way UTF-8 combining characters works continues to give me a headache.
This is a pass at resolving the case, but the logic is unfortunate.
I have a strong feeling that there is a regression from this.

At this point I feel unit tests are necessary.

Perform the combining check after checking white space.
Make the code more flexible by not returning on any invalid character.
An invalid character cannot be a white space, so treat it as not a white space and continue.
The resulting code should be more robust.

2 years agoBugfix: The print functions are not printing the last letter in certain cases.
Kevin Day [Sat, 4 Jun 2022 01:17:40 +0000 (20:17 -0500)]
Bugfix: The print functions are not printing the last letter in certain cases.

The overflow check is testing against >= stop or length.
The equal sign is the problem here.
If the character ends on the stop point or the length, then a whole character would have been printed.
This is not an error and should not result in a return.

Changing >= to > fixes the problem.
Because of this, the additional width check is unnecessary.

2 years agoCleanup: Capitalize first letter of comments.
Kevin Day [Sat, 4 Jun 2022 01:17:05 +0000 (20:17 -0500)]
Cleanup: Capitalize first letter of comments.

2 years agoBugfix: Finish addressing trimmed printing is not working as expected.
Kevin Day [Fri, 3 Jun 2022 03:46:36 +0000 (22:46 -0500)]
Bugfix: Finish addressing trimmed printing is not working as expected.

The previous commit c8c59e16bb4cb7936142bff4815b302ad57138de is incomplete.
Finish addressing the todos and also catch other cases where trimmed printing is not working as expected.

The addition of the combining character checks introduces additional complexity.
This complexity is stronger reason to write unit tests for fl_print.
There is a lot of room for error as a result of this additional logic.

2 years agoBugfix: Trimmed printing is not working as expected.
Kevin Day [Fri, 3 Jun 2022 01:10:07 +0000 (20:10 -0500)]
Bugfix: Trimmed printing is not working as expected.

The length is being used in several functions where an exclusive stop should be used.

There are some improper max length checks.
These are likely the result of a refactor where range checks were replaced with length checks.

In some cases a counter is missing resulting in an infinite loop.

A stop length should be using a >= rather than an == because the incremented width could be greater than one.

2 years agoCleanup: Rename "output" to "stream".
Kevin Day [Fri, 3 Jun 2022 01:09:30 +0000 (20:09 -0500)]
Cleanup: Rename "output" to "stream".

Using the word "stream" is a better and more precise term.
The word "stream" is already in use elsewhere and using this makes the project more consistent.

2 years agoUpdate: Unit tests, fixing minor problems and adding missing functionality.
Kevin Day [Thu, 2 Jun 2022 04:44:06 +0000 (23:44 -0500)]
Update: Unit tests, fixing minor problems and adding missing functionality.

Many of the testfile files have "tests" when they instead should have "test".

The f_print must have a clang specific define passed to work-around a clang problem.

I noticed that I forgot to implement several "returns_data_not" unit test while writing the f_fss unit tests.
Provide these missing unit tests for the f_type_array project unit tests.
This project is my de-facto standard for basing all of my standard data type unit tests and needs to be as correct and up to date as possible.

2 years agoUpdate: Add the status codes.
Kevin Day [Thu, 2 Jun 2022 04:42:19 +0000 (23:42 -0500)]
Update: Add the status codes.

I forgot to add these to a previous commit and almost mixed them in a different commit that they do not belong in.

2 years agoUpdate: Add unit tests for f_fss project.
Kevin Day [Thu, 2 Jun 2022 04:39:40 +0000 (23:39 -0500)]
Update: Add unit tests for f_fss project.

2 years agoUpdate: Fixes and changes resulting from f_fss unit tests.
Kevin Day [Thu, 2 Jun 2022 04:07:33 +0000 (23:07 -0500)]
Update: Fixes and changes resulting from f_fss unit tests.

After reviewing the code I found that the logic is outdated.
The logic was written at the time when I was still far less familiar with Unicode.
The zero-width is being processed in the same way as combining characters.
This needs to change.

After reviewing the combining character logic, I realized that there are bigger problems.
The combining characters, for whatever reason, are processed from right to left.
This breaks normal streaming logic and requires post checks on all variables.
I am amazed that they let such a horrible idea get into the standard and I do not know why.
Regardless, the logic on many of these functions needs to change.
The logic is changed to more properly handle the combining characters and additional more explicit code comment documentation is added.
There will likely need to be follow up changes in the processing code but I am going to leave that for another time.

Change the code to be more consistent with how F_data_not is returned.
Rename functions to more consistently follow the naming strategy used (such as changing "_between" to "_range").
Add f_fss_is_combining() function.
Remove unused and unnecessary f_fss_shift_delimit() function.
Remove unused and unnecessary f_fss_skip_past_non_graph() function.
Update the FSS specifications to better clarify the combining character situation.

The combining character logic implies that I need to return status codes to designate that the return is happening at the start of some processing.
Create several "_start" status codes to address this need.

Fix a bug from a previous commit that is the result of a misplaced regex replace ("alphabeticbetic" should instead be "alphabetic").
Fix a bug where some files where missed when refactoring is_alpha() into is_alphabetic().

2 years agoUpdate: Add Unicode Symbols for currency and modifiers and minor comment clean ups.
Kevin Day [Sun, 29 May 2022 21:03:15 +0000 (16:03 -0500)]
Update: Add Unicode Symbols for currency and modifiers and minor comment clean ups.

The symbols are being added in stages.
Add the currency and modifier symbols.

Update several comments with the todo and fixme notations.

2 years agoCleanup: Remove the parameter checking macros around the function definitions in...
Kevin Day [Sun, 29 May 2022 03:29:56 +0000 (22:29 -0500)]
Cleanup: Remove the parameter checking macros around the function definitions in the unit test.

The tests do not really matter when it comes to file size.
Simplify the test code by not using these macros.

These macros are still used to perform the call.
They just do not need to be around the function definitions.

2 years agoCleanup: Remove stale todo comment.
Kevin Day [Sun, 29 May 2022 03:29:43 +0000 (22:29 -0500)]
Cleanup: Remove stale todo comment.

2 years agoBugfix: Unit test for f_serialize has incorrect dependencies.
Kevin Day [Sun, 29 May 2022 03:29:03 +0000 (22:29 -0500)]
Bugfix: Unit test for f_serialize has incorrect dependencies.

2 years agoUpdate: Add unit tests for f_print.
Kevin Day [Sun, 29 May 2022 03:27:44 +0000 (22:27 -0500)]
Update: Add unit tests for f_print.

I opted to perform the bare minimum needed tests due to the size of this.
Eventually I need to check to see that the correct characters are printed.

2 years agoBugfix: Fix problems exposed by unit tests in f_print.
Kevin Day [Sun, 29 May 2022 03:04:46 +0000 (22:04 -0500)]
Bugfix: Fix problems exposed by unit tests in f_print.

Swap the length a string checks, the length has priority over string.

When performing the character safely prints, the F_utf should only be returned for UTF-8 characters whose width is greater than 1.
The ASCII characters are now no longer returning F_utf.

Some functions are missing the clearerr_unlocked() and ferror_unlocked() calls that are needed for proper fwrite_unlocked() error checks.

Update the documentation comments, adding missing information.

Some of the *_to* functions are not checking if the counter "i" exceeds the length before checking for NULL.

The *_to* functions for *_raw_safely* do not exist and should to be consistent with the regular print functions.
Add the missing *_to*_raw_safely* functions.

Several of the *_to_except* functions are missing the offset parameter which should be passed for consistency with the regular print functions.

Random functions are missing the final print that should exist outside of the loop.
The "total" needs to be checked and if it represents that unprinted data is present, then print that data.

2 years agoFeature: Provide rebuild option in package script.
Kevin Day [Sat, 28 May 2022 02:04:39 +0000 (21:04 -0500)]
Feature: Provide rebuild option in package script.

I'm constantly using clean and build.
Create a single option to do both.

2 years agoCleanup: Restructure f_utf project to make it more manageable.
Kevin Day [Sat, 28 May 2022 01:54:28 +0000 (20:54 -0500)]
Cleanup: Restructure f_utf project to make it more manageable.

2 years agoUpdate: The f_utf punctuation codes.
Kevin Day [Sat, 28 May 2022 00:46:12 +0000 (19:46 -0500)]
Update: The f_utf punctuation codes.

2 years agoBugfix: IKI variable substitution in fake doesn't always expand properly.
Kevin Day [Thu, 26 May 2022 02:56:39 +0000 (21:56 -0500)]
Bugfix: IKI variable substitution in fake doesn't always expand properly.

The commit 85738c3aed3949cc3f26b33e39d28b69dcc6d571 didn't fully solve the problem.

One problem is a copy & paste mistake in the declaration of fake_make_parameter_variable_option_build_s where the wrong define is used.

Make sure to conditionally allocate the arguments array before operating on the "used" position.

The used_content variable should not be needed because the arguments.used should be 0.
After conditional allocation, ensure that the arguments.used is 0 before operation and remove the no longer needed used_content.

When the reserved IKI variables that represent program parameters are used and exist in isolation for their argument, then they should expand as separate variables.
Consider these four examples:
  print 1 parameter:"build"
  print 2 parameter:"build".
  print 3 "parameter:"build""
  print 4 "parameter:"build\" between parameter:"build""

Lets say fake is called with the following "fake make -b /tmp/".
The "print 1" example would have the following parameters:
1) 1
2) -b
3) /tmp/

The "print 2" example would have the following parameters:
1) 2
2) -b /tmp/.

The "print 3" example would have the following parameters:
1) 3
2) -b /tmp/

The "print 4" example would have the following parameters:
1) 3
2) -b /tmp/ between -b /tmp/

The "print 1" expands into 3 parameters because the IKI variable is by itself for that given argument.
The "print 2" expands into 2 parameters because the IKI variable is not by itself for the given argument (It has a period '.' at the end).
The "print 3" expands into 2 parameters because it is quoted and is treated as a single argument.
The "print 4" expands into 2 parameters because it is quoted and is treated as a single argument and the "between" should still be between the two substitutions.

A break is added at the end of one of the loops because that part of the loop is only reached after a match.
When a match is identified, the loop no longer needs further iterations.

2 years agoBugfix: The quotes are always being defined even when there are no quotes in the...
Kevin Day [Thu, 26 May 2022 01:00:40 +0000 (20:00 -0500)]
Bugfix: The quotes are always being defined even when there are no quotes in the FSS read functions.

Do not force either double or single quote by default.
If there are no quotes, then they should be treated as unquoted.

2 years agoUpdate: Additional test settings and add initial coverage support.
Kevin Day [Wed, 25 May 2022 03:15:41 +0000 (22:15 -0500)]
Update: Additional test settings and add initial coverage support.

The goal of the coverage is to support gcov.
I am very unfamiliar with gcov at this time.
The settings are experimental at best.
There will likely be numerous changes relating to gcov in the future as I learn the tool and devised a process to build coverage reports.

2 years agoUpdate: The fake "build" fakefile Object needs to support modes and fix a related...
Kevin Day [Wed, 25 May 2022 02:11:44 +0000 (21:11 -0500)]
Update: The fake "build" fakefile Object needs to support modes and fix a related bug.

The fakefile needs to be able to support being passed custom modes.
The default behavior is preserved with this change.

This requires supporting that an empty string is passed as Content.
It turns out that the quoted empty string is incorrectly being skipped.
Make sure the quoted empty strings are not skipped.

2 years agoCleanup: Existing unit tests, adjust status check behavior.
Kevin Day [Tue, 24 May 2022 04:31:37 +0000 (23:31 -0500)]
Cleanup: Existing unit tests, adjust status check behavior.

I was lazy with the previous behavior and always cleared the error bits when performing the comparison checks.
Change the behavior to properly check the status code for when the error bit is expected and when it is not.

2 years agoUpdate: Add unit tests for f_socket.
Kevin Day [Tue, 24 May 2022 04:31:18 +0000 (23:31 -0500)]
Update: Add unit tests for f_socket.

2 years agoBugfix: Problems exposed by unit tests.
Kevin Day [Tue, 24 May 2022 04:30:52 +0000 (23:30 -0500)]
Bugfix: Problems exposed by unit tests.

Only call strncpy() if socket->name is defined.

Only set socket->id to -1 for f_socket_close_fast_e.
When received EINTR, do set socket->id to -1 for f_socket_close_fast_e.

Be consistent and make length parameter optional for the socket read and write functions.

2 years agoBugfix: Remove extra zero from hex-digit.
Kevin Day [Mon, 23 May 2022 17:20:49 +0000 (12:20 -0500)]
Bugfix: Remove extra zero from hex-digit.

This extra digit results in an always true boolean logic.

2 years agoBugfix: Combining and Width detection for utf8 are not properly printing.
Kevin Day [Mon, 23 May 2022 02:39:27 +0000 (21:39 -0500)]
Bugfix: Combining and Width detection for utf8 are not properly printing.

The wrong data is being passed to utf8_print_combining_or_width().
Change the behavior to send the correct string to the function.
Move the error printing to a single function and use this function in all such cases.

2 years agoCleanup: Incorrect function documented in comment.
Kevin Day [Mon, 23 May 2022 02:11:43 +0000 (21:11 -0500)]
Cleanup: Incorrect function documented in comment.

2 years agoCleanup: Rename bytecode to bytesequence.
Kevin Day [Mon, 23 May 2022 02:07:07 +0000 (21:07 -0500)]
Cleanup: Rename bytecode to bytesequence.

The term "bytecode" already exists and is used for a slighty different purpoe (representing compiled or partially compiled data).
This is a different context.
To avoid using the term improperly, switch to a more proper term "bytesequence" (as one word).
A byte sequence is a term representing a sequence of bytes.
This is more specific than binary and effectively emphasis that this is in regards to bytes.
Avoiding the term binary, however correct or not the term may be, helps avoid confusion due to "binary" and "text" data being considered two separate things.

2 years agoCleanup: Use macro_f_string_static_t_initialize2() with macro_f_string_static_t_initi...
Kevin Day [Mon, 23 May 2022 01:51:29 +0000 (20:51 -0500)]
Cleanup: Use macro_f_string_static_t_initialize2() with macro_f_string_static_t_initialize().

Using macro_f_string_static_t_initialize2() was the old way and is now deemed incorrect.
The macro_f_string_static_t_initialize2() applies the length parameter to both the used and size.
For static strings the size is always 0 because nothing is dynamically allocated.
Therefore, using macro_f_string_static_t_initialize2() for static string initialization is incorrect.

2 years agoUpdate: Tweak endiannes for f_utf_char_t.
Kevin Day [Mon, 23 May 2022 01:47:53 +0000 (20:47 -0500)]
Update: Tweak endiannes for f_utf_char_t.

I continue to forget that the f_utf_char_t is a big-endian format regardless of the host endiannes.
I then end up comparing the endiannes logic to normal operations and find discrepancies.
I waste a good bit of time to ultimately realize that the f_utf_char_t is not in host byte order.

Update the comments to better represent this situation.
I also noticed that the big endian bitwise operations are going in the wrong directory.
I could be wrong, but I think I need to do a left shift rather than a right shift.
Or perhaps, this only needs to be done on a big-endian system?
I need to test this logic on a big endian system.

2 years agoFeature: Add missing functionality allowing the utf8 program to convert back to binar...
Kevin Day [Mon, 23 May 2022 01:45:31 +0000 (20:45 -0500)]
Feature: Add missing functionality allowing the utf8 program to convert back to binary data with invalid codepoints.

Even when there are invalid codepoints produced, it should be possible to convert the entire output back to the original data.
This is possible because the codepoint output by default still prints the invalid data as a hex-digit representing up to 4 bytes of data.
The combining and width parameters are also supported.

2 years agoUpdate: Use F_utf_not instead of F_utf and other resulting changes.
Kevin Day [Sun, 22 May 2022 03:06:56 +0000 (22:06 -0500)]
Update: Use F_utf_not instead of F_utf and other resulting changes.

The F_utf_not is semantically more correct than F_utf when returning an error for an invalid UTF-8 sequence.
Use F_utf_fragment where appropriate as well.

Updating all of the appropriate comments revealed some documentation and code structure problems in the fss projects.
These are cleaned up as well.

2 years agoUpdate: Project f_utf.
Kevin Day [Sat, 21 May 2022 21:22:13 +0000 (16:22 -0500)]
Update: Project f_utf.

While investigating the utf8 program, I looked into the f_utf project and found that it is still very much lacking.
At some point in the process of me writing this, the Unicode 14 was released.
I started the process of updating parts of the code and have made it as far as Gujarati with this commit.

Remove unused functions.
Add new functions for detecting if something is a superscript or a subscript.

Update the comments in the private functions to make it explicitly clear when a particular private function expects that only characters of width 2 or greater are provided.

There are some "todo" comments that need to be addressed before the stable release.
I'm expecting another release candidate at this point and so I am pushing off some of the Unicode updates onto after the next release candidate.

I noticed that the unit tests for f_utf only address the structures.
While this is disappointing it does save me the effort of having to write more unit tests for the newly added functions.

2 years agoRegression: The utf8 program is only returning space when converting file from codepoint.
Kevin Day [Sat, 21 May 2022 02:09:54 +0000 (21:09 -0500)]
Regression: The utf8 program is only returning space when converting file from codepoint.

The non-space characters are always being defined as invalid.

2 years agoCleanup: Remove no longer being used 'u' from macros.
Kevin Day [Sat, 21 May 2022 02:08:31 +0000 (21:08 -0500)]
Cleanup: Remove no longer being used 'u' from macros.

2 years agoCleanup: Stale code, improve usage, replace macros.
Kevin Day [Fri, 20 May 2022 04:50:36 +0000 (23:50 -0500)]
Cleanup: Stale code, improve usage, replace macros.

The primary focus of this commit is to remove stale code exposed by compilers.
This pass I used clang with -Wall.

Fixed some usage cases where the variables can be replaced with other variables.

I happened to notice some macros didn't need to exist and added the appropriate methods.
I did not search for other cases like this.
I only fixed what happened to be in front of me at the time.

2 years agoSecurity: Add missing NULL at the end of string.
Kevin Day [Fri, 20 May 2022 04:07:28 +0000 (23:07 -0500)]
Security: Add missing NULL at the end of string.

I was seeing "-Wno-missing-bracess" and "-Wno-logical-op-parenthese".
This looks exactly like invalid sizes.
I could not find the code that misplaces the NULL but in the process I found a different place where a NULL is in fact missing.
This adds the missing NULL.
I guess if one looks hard enough, then they will find what they are looking for.

It turns out that my original problem is actually two typos in a configuration file.

2 years agoUpdate: Add unit tests for f_serialize.
Kevin Day [Thu, 19 May 2022 05:58:02 +0000 (00:58 -0500)]
Update: Add unit tests for f_serialize.

2 years agoUpdate: Rewrite the f_serialize functions.
Kevin Day [Thu, 19 May 2022 05:56:28 +0000 (00:56 -0500)]
Update: Rewrite the f_serialize functions.

The code is outdated and needs to be updated with the current practices.
I remember being uncertain on what to name several of these functions.
I didn't want an f_unserialize function because that would require a separate project (even if the naming makes sense).
Perhaps I may do that in the future, but for now just use the words "from" and "to".

2 years agoSecurity: The realpath() calls malloc() and free() is not called (memory leak).
Kevin Day [Wed, 18 May 2022 02:48:50 +0000 (21:48 -0500)]
Security: The realpath() calls malloc() and free() is not called (memory leak).

I change the code and didn't realize that realpath() conditionally calls malloc().
When I change th code to pass a variable initialized to 0, I ended up triggering realpath() to call malloc().
This results in a memory leak.

Instead, pass a statically allocated array.

2 years agoCleanup: Replace the number 0 with the word zero.
Kevin Day [Tue, 17 May 2022 23:14:43 +0000 (18:14 -0500)]
Cleanup: Replace the number 0 with the word zero.

2 years agoCleanup: Rename pictures that have logo in the name twice.
Kevin Day [Tue, 17 May 2022 03:07:53 +0000 (22:07 -0500)]
Cleanup: Rename pictures that have logo in the name twice.

These should be named icon not logo.

2 years agoUpdate: Specifications.
Kevin Day [Tue, 17 May 2022 02:47:06 +0000 (21:47 -0500)]
Update: Specifications.

Started using IKI format in the specification files.
Made changes and performed fixes as I noticed them while copying the specifications to the website.

2 years agoBugfix: The iki_read program is not handling verbosity correctly.
Kevin Day [Mon, 16 May 2022 00:25:15 +0000 (19:25 -0500)]
Bugfix: The iki_read program is not handling verbosity correctly.

The newline should be printed at the end of the program unless in quiet mode.
This fails for two reasons:
1) The conditional checks before printing are wrong.
2) The quiet parameter is at the wrong position resulting in it being mixed up with the "no color" parameter.

2 years agoFeature: The iki_read program should support wrapping a variable value.
Kevin Day [Sun, 15 May 2022 23:49:47 +0000 (18:49 -0500)]
Feature: The iki_read program should support wrapping a variable value.

One of the original design intentions of the IKI standard is to allow for substitution.
That substitution includes wrapping text with something like HTML markup.
The current design of iki_read falls short here.

While the substitution can be performed, the wrapping while preserving the existing value is not performed.
For example consider the following:
  emphasis:"Some message."

This should be substituted with the HTML5 "<em>" tag.
The substitute parameter requires knowing the value.
The replace parameter also requires knowing the value.

The emphasis HTML5 markup needs to be prepended and appended without having to know every single value.

To solve this, the -W/--wrap option is now available.
This is a 3 parameter option that acts similar to the -r/--replace parameter.
However, it will instead accept a "before" and "after" representing the before and after strings.
Either the before or after string may be an empty string.

The design of this feature re-utilizes existing structures.
These structures have context in their names that do not match "before" and "after".
This can be confusing, but this is considered an inconvenince at this time.
The goal is to keep the changes simple if at all possible with a stable release around the corner.
I also do now know what words to use to share between the different types without creating a new one to make such a change.

This feature is necessesary to ensure completeness with the original intent and design of both the IKI standard and the iki_read program.

2 years agoBugfix: Previous commit overlooked the counter resulting in some replacements not...
Kevin Day [Sun, 15 May 2022 22:49:51 +0000 (17:49 -0500)]
Bugfix: Previous commit overlooked the counter resulting in some replacements not being processed.

Oops! This is what I get for rushing this out with little testing and review.

The problem here is that the "+3" is for the 3 parameter option "--substitute".
The 2 parameter option "--replace" must instead use "+2".

2 years agoFeature: The iki_read program should support a more generalized substitution process...
Kevin Day [Sun, 15 May 2022 21:31:11 +0000 (16:31 -0500)]
Feature: The iki_read program should support a more generalized substitution process called "replace".

In the distant past I mixed up having only two or three substitution parameters.
After getting confused, I decided to just have a 3 argument substitution.
The three argument substitution only substitutes if both the variable name and the variable value match.

This is great but it doesn't follow the completeness theorem.
The iki_read should also handle the general case in addition to the specific case.

Provide a two argument substitution called "replace" that handles the more general case.
When any variable name matches the given replace parameter, then the variable value is replaced for all matching variable names regardless of the existing variable value.

Fix small mistakes in the IKI specification.
I needed to refer to the "variable value" with this change and "variable variable" is simply not the correct way.
Use "variable names" rather than "object names" to be more consistent and clear in this regard.

2 years agoUpdate: Add cmocka specific environment variables to the testfiles.
Kevin Day [Sun, 15 May 2022 16:06:46 +0000 (11:06 -0500)]
Update: Add cmocka specific environment variables to the testfiles.

These environment variables are commented out by default.
The "define" "settings" Object defines the environment variable.
The "environment" "settings" Object exposes that environment variable to any called programs.

2 years agoUpdate: Add unit tests for fll_status_string.
Kevin Day [Sun, 15 May 2022 05:09:40 +0000 (00:09 -0500)]
Update: Add unit tests for fll_status_string.

2 years agoBugfix: Fix problems exposed by unit tests.
Kevin Day [Sun, 15 May 2022 05:09:03 +0000 (00:09 -0500)]
Bugfix: Fix problems exposed by unit tests.

2 years agoUpdate: Add unit tests for f_status_string.
Kevin Day [Sun, 15 May 2022 04:39:44 +0000 (23:39 -0500)]
Update: Add unit tests for f_status_string.

2 years agoBugfix: Problems exposed by unit tests and do some relevant code style clean up.
Kevin Day [Sun, 15 May 2022 04:38:49 +0000 (23:38 -0500)]
Bugfix: Problems exposed by unit tests and do some relevant code style clean up.

2 years agoUpdate: Unit tests, fixing minor mistakes and add missing tests.
Kevin Day [Sun, 15 May 2022 03:19:50 +0000 (22:19 -0500)]
Update: Unit tests, fixing minor mistakes and add missing tests.

While these tests are not expected to be thorough at this time, there are several checks that should still be and can easily be made.

Some of the test files have incorrect comments.

Order test functions alphabetically.

2 years agoBugfix: Add missing return value handling in f_fss.
Kevin Day [Sun, 15 May 2022 00:59:57 +0000 (19:59 -0500)]
Bugfix: Add missing return value handling in f_fss.

2 years agoUpdate: Add unit tests for f_limit.
Kevin Day [Sun, 15 May 2022 00:59:38 +0000 (19:59 -0500)]
Update: Add unit tests for f_limit.

2 years agoUpdate: The f_limit project is out of date.
Kevin Day [Sun, 15 May 2022 00:57:19 +0000 (19:57 -0500)]
Update: The f_limit project is out of date.

Writing unit tests revealed that the f_limit project does not operate like the latest code.
Restructure and rewrite f_limit to be consistent with the latest practices and designs in the rest of the project.

Other projects utilizing this are also updated.

2 years agoBugfix: Incorrect and missing initializer macros in f_fss.
Kevin Day [Sat, 14 May 2022 21:34:48 +0000 (16:34 -0500)]
Bugfix: Incorrect and missing initializer macros in f_fss.

2 years agoRegression: Fix mistake in variable name of controller program.
Kevin Day [Sat, 14 May 2022 20:24:17 +0000 (15:24 -0500)]
Regression: Fix mistake in variable name of controller program.

When I added a missing variable to the function, I somehow overlooked a compiler error/warning.
This should be "global.main" and not "main".

2 years agoUpdate: Add new status codes and fix problems with existing ones.
Kevin Day [Sat, 14 May 2022 19:42:46 +0000 (14:42 -0500)]
Update: Add new status codes and fix problems with existing ones.

The API will be frozen on the stable release.
There are upcoming changes in the next development cycle that will focus on networking.
Provide additional status codes that will be used in networking to make transition and compatibility simpler and easier.

There are also plans in adding init support to the controller program.
Operations such as halt and terminate become necessary.

While working on this I noticed this introduces a discrepancy between "terminate" and "terminated".
The "terminated" is meant to focus on buffers, such as a terminated string.
To fix this conflict, I decided to favor the practice of trying to always use present tense.
This means replacing "terminated" with a present tense word.
I chose "end".
There already is an F_end, so break out a new status section and move all of the newly minted "end" types into that.

While making these changes I noticed and fixed a few problems.

There is both F_warn and F_warning.
Remove F_warn in favor of F_warning.

Add these new status codes (and their _not compliments):
- F_break
- F_desire
- F_discard
- F_drop
- F_exit
- F_halt
- F_keep
- F_need
- F_restart
- F_restore
- F_revert
- F_skip
- F_store
- F_terminate
- F_want
- F_wish

The F_string_too_large and F_string_too_small checks are incorrectly returning F_too_large_s and F_too_small_s, respectively, when they instead should be returning F_string_too_large_s and F_string_too_small_s.

2 years agoBugfix: Use '%Q' rather than '%S' when printing error.prefix and add missing error...
Kevin Day [Sat, 14 May 2022 13:56:00 +0000 (08:56 -0500)]
Bugfix: Use '%Q' rather than '%S' when printing error.prefix and add missing error.prefix.

2 years agoCleanup: Use standard name when printing message.
Kevin Day [Sat, 14 May 2022 13:50:54 +0000 (08:50 -0500)]
Cleanup: Use standard name when printing message.

2 years agoBugfix: The FSS object write functions don't handle all newlines.
Kevin Day [Sat, 14 May 2022 13:40:31 +0000 (08:40 -0500)]
Bugfix: The FSS object write functions don't handle all newlines.

There are several cases where the newline character in the object is not being properly treated as an error and returned.

2 years agoCleanup: Add U+000A Unicode codepoint in error message.
Kevin Day [Sat, 14 May 2022 13:06:40 +0000 (08:06 -0500)]
Cleanup: Add U+000A Unicode codepoint in error message.

2 years agoRefactor: Rename -R/--raw to -O/--original in programs.
Kevin Day [Sat, 14 May 2022 00:12:13 +0000 (19:12 -0500)]
Refactor: Rename -R/--raw to -O/--original in programs.

The help for this parameter reads "Print with the original quotes and escapes.".
Using the word "original" simply makes more sense that "raw".

Now that there is a reason to use "raw" for other purposes, refactor "raw" into "original" so that "raw" can be made available for defining.

2 years agoUpdate: Add unit tests for f_pipe.
Kevin Day [Fri, 13 May 2022 23:27:25 +0000 (18:27 -0500)]
Update: Add unit tests for f_pipe.

2 years agoUpdate: Improvements exposed when writing unit tests.
Kevin Day [Fri, 13 May 2022 23:26:19 +0000 (18:26 -0500)]
Update: Improvements exposed when writing unit tests.

Make sure to call memset on structure.
Minor style clean ups.

2 years agoUpdate: Add unit tests for f_path.
Kevin Day [Fri, 13 May 2022 06:26:33 +0000 (01:26 -0500)]
Update: Add unit tests for f_path.

2 years agoBugfix: Fixes to f_path of problems exposed by unit tests.
Kevin Day [Fri, 13 May 2022 06:21:02 +0000 (01:21 -0500)]
Bugfix: Fixes to f_path of problems exposed by unit tests.

Remove unnecessary variable.
Remove unnecessary if condition.

Change while loop to be more robust in .
Check the path.used before checking path.string in case path.used is 0 (this should be safer).

Update function documentation.

The logic in private_f_path_real() is incorrect.
The realpath() function returns a string (char *) rather than an integer.

Also do some code clean up.

2 years agoSecurity: Add missing parameter checks and rename "data" to "custom".
Kevin Day [Fri, 13 May 2022 00:11:24 +0000 (19:11 -0500)]
Security: Add missing parameter checks and rename "data" to "custom".

Using "data" as the variable name for the "custom" property is confusing and can lead to mistakes.
Use "custom" to directly match that this is the "custom" property rather than the "data" property.

2 years agoUpdate: Add missing state.flag assignment in f_state_t_clear macro and fix syntax...
Kevin Day [Fri, 13 May 2022 00:11:04 +0000 (19:11 -0500)]
Update: Add missing state.flag assignment in f_state_t_clear macro and fix syntax practice in initializer.

2 years agoSecurity: Wrong variable passed resulting in invalid read and writes in Fake.
Kevin Day [Fri, 13 May 2022 00:09:33 +0000 (19:09 -0500)]
Security: Wrong variable passed resulting in invalid read and writes in Fake.

2 years agoBugfix: Environment variables in fake are not being expanded.
Kevin Day [Thu, 12 May 2022 05:30:59 +0000 (00:30 -0500)]
Bugfix: Environment variables in fake are not being expanded.

The if condition "if (data_make->setting_make.load_build) " should no longer be relevant.
The environment variable should not be tied to load_build.

Change the fl_string_dynamic_partial_compare() checks to fl_string_dynamic_partial_compare_string() to avoid needing to define a custom range.

2 years agoBugfix: The fake program should not require the data directory when explicit fakefile...
Kevin Day [Thu, 12 May 2022 02:49:04 +0000 (21:49 -0500)]
Bugfix: The fake program should not require the data directory when explicit fakefile or settings files are specified.

Set or reset the validate_parameter_directories check as appropriate when calling 'clean' or 'skeleton' operations.

Make the parameters_required check contingent on the presence of the parameters --fakefile and --settings.
When these are specified, do not even bother checking for the data directory at all.

2 years agoUpdate: Add missing function f_path_is_absolute() and fix existing f_path_is_*()...
Kevin Day [Thu, 12 May 2022 02:42:44 +0000 (21:42 -0500)]
Update: Add missing function f_path_is_absolute() and fix existing f_path_is_*() functions.

The f_path_is_absolute() function, being the compliment of f_path_is_relative() is now added.

I noticed multiple problems when looking at this code.
- The f_path_is_relative() and f_path_is_relative_current() functions are not checking that the max length is reached before comparing.
- the f_path_is_relative_current() is not incrementing the counter when attempting to check for the next character resulting in invalid results.

The f_path project clearly needs unit testing.
I intend to write unit tests and fix problems found before the next stable release is made.

2 years agoUpdate: Utilize the state.flag to allow for fss read to not fail out on invalid UTF...
Kevin Day [Wed, 11 May 2022 05:38:37 +0000 (00:38 -0500)]
Update: Utilize the state.flag to allow for fss read to not fail out on invalid UTF-8 code sequence and fix naming problems.

One of the original goals of the FLL project is to achieve fail-through functionality.
Knowing that this is a lot of work, I have ignored a lot of situations where I can implement fail-through and simply performed fail-out or fail-over.

With the upcoming stable release, I believe that this must handle bad data files.
This adds the option to conditionally change the behavior between fail-through and fail-out for the fss read functions and related for invalid UTF-8 code sequences.
The default behavior is now changed from fail-out to fail-through.

This took longer than I hoped.
I will need to do additional reviewing of this code before the stable release is ready.
I also realized that I need to support raw printing of data in the fss read functions as well (and that means changing the existing -r/--raw parameter).

This also fixes the following naming problems:
- fl_fss_apply_delimit() should be f_fss_apply_delimit().
- fl_fss_apply_delimit_between() should be fl_fss_apply_delimit_between().

2 years agoUpdate: Utilize the state.flag to allow for iki read to not fail out on invalid UTF...
Kevin Day [Wed, 11 May 2022 03:19:54 +0000 (22:19 -0500)]
Update: Utilize the state.flag to allow for iki read to not fail out on invalid UTF-8 code sequence.

One of the original goals of the FLL project is to achieve fail-through functionality.
Knowing that this is a lot of work, I have ignored a lot of situations where I can implement fail-through and simply performed fail-out or fail-over.

With the upcoming stable release, I believe that this must handle bad data files.
This adds the option to conditionally change the behavior between fail-through and fail-out for the f_iki_read() and related for invalid UTF-8 code sequences.
The default behavior is now changed from fail-out to fail-through.

2 years agoUpdate: The f_utf_is_* functions should be more specific on return state for F_failure.
Kevin Day [Wed, 11 May 2022 03:17:38 +0000 (22:17 -0500)]
Update: The f_utf_is_* functions should be more specific on return state for F_failure.

Rather than returning F_failure, return F_complete_not_utf.

2 years agoBugfix: Invalid read in f_file_stream_read_until().
Kevin Day [Wed, 11 May 2022 03:08:59 +0000 (22:08 -0500)]
Bugfix: Invalid read in f_file_stream_read_until().

The file.size_read variable is incorrectly being used when instead the buffer_size variable is intended to be used.