Regression: Follow up to invalid type used when allocating string, f_string_t should instead be f_char_t.

I missed the save button on this file and this change did not get committed.

Progress: Begin simplifying memory design, starting with f_number_unsigned_t array structures.

I find that the use of custom per-type functions to simplify the usability of the library.
I also find that this increases the complexity and size of the code base.

I believe that over time this will become more complex and even harder to maintain.
I have decided to start changing the design to go back to simple allocation methods that just require more variables.
This simplifies the code and in theory will make it faster by having fewer function calls on the stack.
This may even reduce the resulting binary size (and I hope it does).

The down side is that the users of the library will once more have longer function call lines.

This commit is the beginning of the redesign process.

I also added the "append" and "append all" methods for f_memory_array calls.

Regression: Invalid type used when allocating string, f_string_t should instead be f_char_t.

I noticed odd differences between the 0.6.x and the 0.7.x in terms of memory usage.
I discovered this to be due to a bug where the f_string_t is being used when the type should be instead f_char_t.
This is likely a bug introduced in some past refactor.

Cleanup: Surround constant variable in a scoped block.

Progress: Begin further re-designing of the memory logic.

This adds the append and append all functions, respectively.
This adds a paramete check to see if width is 0.

Update: Add comment in Featureless Make code to designate changes the must be done.

The use of main->program.parameters.arguments should no longer happen.
This is the old behavior and it looks like it has not been completely migrated.

This is wrong an can even have invalid read sizes.

Feature: Add f_simple_packet_t for holding the packet data.

The f_simple_packet_range_t holds ranges within a string.

This f_simple_packet_t holds the actual values for the packet rather than ranges mapped to some string.

I am omitting unit tests at this time because I am going to do some additional changes involving the allocation and deallocation code project wide.

Bugfix: The socket id is not the same descriptor for reading.

When binding and listening on some socket id, the accept call on that socket creates a new descriptor.
That descriptor is not the same as the original socket id.

Add a new id_data to the socket structure to represent the data transfer file descriptor that is created via the accept() call.
Add a new f_file_close_id() to close the file by the id in case the f_file_t is not available or in use.

Feature: Add f_time project, adding functions for safely handling struct timespec.

This adds f_time_spec_millisecond() and f_time_spec_nanosecond() for safely handling overflow and underflow when using standard FLL f_number_unsigned_t numbers.

Unit tests are added.

A comment in the controller prrogram is added to designate that the controller-specific functions should be updated to call these functions.

Cleanup: Style in this f_print source file.

Update: Wrap the static inline functions private_inline_f_print_to_error in custom disable macros.

As an exception case, wrap the private_inline_f_print_to_error() in _di_private_inline_private_f_print_to_error_ and _di_private_inline_f_print_to_error_ as appropriate.
Using a define condition on all of the possible cases would be difficult to manage.

An explicit define is simpler and easier in this exceptional case.

Cleanup: Reduce usage of static in functions.

Remove the static from variables in functions that are not likely to be called multiple times and are not expected to be recursive.

I am backing off the experiment into this optimization for now.

Cleanup: Add extra parenthesis to prevent warning from clang.

Update: Stand alone Featureless Make build settings.

Cleanup: Remove the FSS Embedded List Program.

This embedded_list in fss_read should be used going forward.
The embedded_list in fss_read is not working yet and is just a copy of extended_list.
I will eventually get around to updating the embedded_list in fss_read to work as intended and expected.

Update: Set the pipe flag consistely across programs.

Use the same practice of designating the pipe as a main flag, even if the program isn't using the input pipe.

For the fss_read programs that use the word "pipe" in a main flag are renamed to "pipe" "format".

Update: Control program to latest design practices, migrating from 0.6.x to 0.7.x design.

This gets the control program to compile under the latest design practices.

I did not to an in depth review as this program does not currently do anything.
Once the controller program is updated I will eventually come back to this.

Update: Move standard console parameters enumeration into a standard enumeration set and provide appropriate functions.

This should make the commonly used code slightly easier to maintain.

The control project is omitted because I am in the process of migrating it.
The migrated version will contain the appropriate changes from this.

Cleanup: Byte Dump program defines and comment.

Bugfix: The Byte Dump program prints extra new lines.

Migrating from the old way to the new way results in extra new lines because of the first and last new line being printed.
Remove the extra new line being printed but still print in all other cases.

Bugfix: Byte Dump program 6-space string has only 5 spaces.

Add missing space.

Feature: Add f_select_signal() and update unit tests.

This adds the missing pselect() wrapper, f_select_signal().

Workaround: Clang compiler problems, add _clang_not_a_compile_time_constant_workaround_ define in Byte Dump program.

CLang fails to handle this well and errors out with: "initializer element is not a compile-time constant".

Provide a custom define _clang_not_a_compile_time_constant_workaround_ to be used as a work-around for the problematic CLang compiler.

Bugfix: Add missing return statement in Byte Dump function.

Update: Remove unused fl_signal and fl_signal_state_interrupt.

The fl_signal_state_interrupt() function is not used and is replaced by fll_program_standard_signal_received().

The entire fl_signal can be and is removed.

Bugfix: Byte Dump program binary mode is printing wrong number of padding.

Cleanup: Set named 0 enumeration in Byte Dump program.

Update: Relocate several static stings into single location for Byte Dump program.

Bugfix: Fix new lines when printing help.

I noticed a couple of programs are missing the correct new lines when printing the help.

Update: Minor fixes and consistency changes.

Fix some small things that I noticed while I was migrating the byte_dump program.

Update: Byte Dump program structure, following newer standards.

I noticed two things while working on this that need to be addressed for all programs.
1) When using threads, the signal for piping data (such as byte_dump /bin/bash | head) does not trigger.
2) The color code modes are not being processed correctly.

These observed bugs will be fixed in some other commit.

Update: Finish memory array related changes.

There are still the append() and append_all() function to change but I am going to leave that for another time.
This finishes the recent set of Progress commits addressing the memory array related code.

I have observed a notable reduction in binary size but the primary goal here is reduction of duplicate code and simplification of code.

I should eventually be able to re-introduce macros to map these helper functions back to the base array type to reduce binary size by a significant amount.
This should only be done once the API has stabilized more.

Progress: Continue memory array related changes.

I noticed that I could also implement f_memory_array_append() and f_memory_array_append_all() methods if I used memcpy() and a multiplication over the type size.

This:
    const f_status_t status = f_memory_array_increase_by(source.used, sizeof(f_socket_t), (void **) &destination->array, &destination->used, &destination->size);
    if (F_status_is_error(status)) return status;

    for (f_number_unsigned_t i = 0; i < source.used; ++i) {
      destination->array[destination->used++] = source.array[i];
    } // for

    return F_none;

Could instead become:

    const f_status_t status = f_memory_array_increase_by(source.used, sizeof(f_socket_t), (void **) &destination->array, &destination->used, &destination->size);
    if (F_status_is_error(status)) return status;

    memcpy(destination->array, source.array, sizeof(f_socket_t) * source.used);

    return F_none;

This could then be logic could then be moved into something like f_memory_array_append_all(), further simplifying the code into:
    return f_memory_array_append(source.array, source.used, sizeof(f_socket_t), (void **) &destination->array, &destination->used, &destination->size);

I need to think of the function structure that I want and so I only decided to comment in this commit.
Such a change is not implemented yet but may be done at some later time.

Progress: Continue memory array related changes.

Progress: Continue memory array related changes.

This converts a lot of the projects to use the f_memory_array_*() functions.

Break the f_thread private functions into their own files.

Progress: Continue memory array related changes, replacing the f_memory_structure entirely.

The f_memory_array set of functions now fully replaces the f_memory_structure functions and related macros.
I think those structure macros are nifty but they are not in use enough anymore and the f_memory_array structure is what I am going forward with.

There is still an enormouse amount of changes to do.

Progress: Add memory array allocation functionality and begin to use it where possible.

The pattern for handling the standard array structure is now well established.
Provide a memory array functionality for handling the most basic form.

This should reduce repeated code where possible.
It would be interesting if this reduces the size of the resulting binary but with inline optimizations it may be possible that this actually increases the size of the resulting binary.

Update the comments and tests.

There are a lot of files that will need to be updated to use these new functions.
Most of these are in the f_type_array project.

Update: Add FSS Simple Packet related functionality and remove delimit and comment types.

The FSS Simple Packet functionality is added, to provide for processing of the FSS Simple Packet.
The f_fss_simple_packet_ranges_t represents a set of ranges for designating where/how the packet is defined within some string.

The delimit and comment types are just helper type definitions for existing number types.
Simplify the design by getting rid of those, replacing them with the type that they effectively represent.
A lot of code is updated to reflect this change.

Feature: Add specification FSS-0010 "Encrypted Simple Packet".

This is an encrypted variation of the FSS-000F "Simple Packet" specification.
The third control bit (from the left) represents that the Simple Packet is encrypted.

The requirements for the format of the actual contents of Payload Block is further relaxed due encrypted data needing to be private (for what should be obvious reasons).

Update: FSS-000f specification, adding reasoning and improve the wording.

Add a simple paragraph about the reason why the size bit includes the Control Block and the Size Block in its size representation.

Fix a mistake in the words where the standard has "...of that that...".
The back to back use of "that" is confusing and probably a typo.

Cleanup: Remove extra space in documentation comment of the standard.

This does not alter the standard itself and so the version date is left unchanged.

Cleanup: Documentation comments regarding structures.

Add "Properties: " and related styling to the documentation comments for type definition structures.

Fix capitalization and other formatting issues that I noticed.

I did not focus on any of the programs.
However, some program related files happened to be open and so I updated those out of convenience.
A follow up commit will be needed to address the programs and any other files in which I may have missed in this pass.

Update: Specifications, adding explicit version date and change license line format.

Add a version, which contains the date of the last change to each of the specifications.
There will eventually be a specification version number such as "1.2.3".

The license line in the comments at the top of each specification is of this form: "license: ".
Remove the colon so that the comment, if extracted, could be processed as something simple like FSS-0000.

Update: Specifications, replacing "emphasis" with "italic".

Update: FSS-000F (Simple Packet) Specification.

I have decided that the endianness bit must be first.
The String/Binary bit is now the second bit.
The remaining 6 Control bits are left undefined.

Do some cleanup up in the text.
Improve the language.
Make sure "italic" is used instead of "emphasis".

Feature: Add fll_program_print_error_parameter_integer_less_than() and fll_program_print_error_parameter_integer_more_than().

The less than and more than (greater than) are likely a common enough check to create standard functions for.

Update: Conversion should return F_number_positive and F_number_negative without error bit and do some clean up.

Do not treat F_number_positive and F_number_negative as an error.
The "unsigned" concept should be internal to the function and not whether or not the user input has a sign.

If F_number_negative is returned, the value can still be of type f_number_unsigned_t.
This is because the return status designates the effective sign.
The caller can then decide whether or not this is an error.

Update: Use status_signal instead of status_thread.

The status_thread is being used specifically for the signal thread.
Change the standard practice to have this renamed to status_signal.

This then helps make other status names make more sense.
Rather than have status_thread, status_thread_a, and status_thread_b, instead have status_signal, status_a, and status_b.

Cleanup: Use present tense in documentation comments regarding F_interrupt.

Update: The build disable files and stand alone setting files.

Update: Add poll enumerations for f_poll_t.

This also includes a few enumerations based on GNU poll flags.

Cleanup: Fix socket documentation comment, where type is the actual socket type.

The socket.type is not the address type, it is the socket type.

The socket type is something like SOCK_STREAM or f_socket_type_stream_e.

Update: Network additions and improvements, minor documentation comment clean ups.

Add f_network_is_ip_address() along with appropriate unit tests.
The unit test are not as thorough as they could be.
I did not fully review the IPv6 standard, which has a rather absurd design.
There are likely things that I do not yet know about that are not supported.
Specifically do not support port number on any IPv6 address without brackets.
(The IPv6 address notation standard is horribly written.)

Update: Add F_network_version_four, F_network_version_four_not, F_network_version_six, and F_network_version_six_not status codes.

The IPv4 and IPv6 situation is common enough to merit explicit status codes.

Cleanup: Conversion header comments.

Cleanup: Documentation comments regarding "This alters setting.status" in programs.

In such programs, this is comment is no longer accurate.

Update: Socket bind functions, socket address family enumeration, and rename max_backlog.

Change the f_socket_bind_local() bind function to not call memset() and use the new address structure.
Add f_socket_bind_inet4() and f_socket_bind_inet6() bind functions in a similar way to f_socket_bind_local().

Update the appropriate documentation.
Add the appropriate unit tests.

The name max_backlog for f_socket_listen() is better named backlog_max as per the FLL style practices.

Update: The f_socket enumerations and comments, the f_socket_t structure, the f_socket build settings, and the f_socket unit tests.

The _di_f_socket_protocol_e_ enumerations are missing the trailing "_e" in the name.

As stated in a previous commit (90d0f47999fd5615f2f3550d688f63d49e590482), update the documentation comments for the f_socket_protocol_family_e enumerations.

Change the ordering of properties in the f_socket_t to have length property after the address property.

Set the default string for the name property to be f_string_static_t_initialize instead of f_string_empty_s.

Disable the braces warning in f_console.
The brace usage here is correct, and so the warning is disabled.

Fix the unit tests as they were not updated after changes in the previous commit (90d0f47999fd5615f2f3550d688f63d49e590482) and in this commit.

Cleanup: Use the signal check functions in progams.

There already are signal check programs here.
Use those instead of manually performing the signal check actions.

Update: Socket improvements regarding structure and styling.

The f_socket_t structure has "struct sockaddr" as a pointer.
This means mass allocation becomes less practical due to needing to create the actual "struct sockaddr" as needed.
Implement an f_socket_address_t union that accepts the cost of less memory efficiency to improve the usability.

This now allows for mixed types to be stored in the array without needing complicated code to manage different arrays.
The caller will have to be responsible for properly setting the appropriate structure as needed.

Organize the code styling to be more consistent with alphabetical ordering.
I likely have some of the comments to update as well (a follow up commit will be necessary).

Update: Socket file settings, use enumerations rather than defines, and add family enumerations.

This expands the socket family and makes their use more correct and consistent with existing ones.

The socket domain is renamed where possible to be protocol family or address family.

Bring in a lot of the existing family type codes (both protocol family and address family).

Update: Add missing fakefiles for stand alone builds.

Update: The defines in cygwin.txt documentation are out of date.

Cleanup: Add "write" to the open modes involving write.

The "append" and "truncate" are not verbose enough.
Prepend "write_" to each of these to make the purpose and intent more obvious.

Update: The stand alone Featureless Make config.h.

Re-synchronize this file with all of the latest changes.
The stand alone build should now compile again.

Cleanup: There is no f_print_except_dynamic_in_safely() anymore.

There are no existing references to f_print_except_dynamic_in_safely() so remove the disable macro.

I don't remember removing f_print_except_dynamic_in_safely() so hopefully it is intentionally removed.

Update: Add f_sockets_t, add f_socketss_t, and other f_socket changes.

The socket.name is better as an f_string_static_t.

Fix the outdated and incorrect documentation comments for f_socket_t.

Add additional macro functions for initializing the f_socket_t.

Cleanup: Don't bother with != for existence modulus checks.

The 'X % 2' and 'X % 3' will be non-zero with they are not divisible by the given unit.
The '!= 0' check is not necessary.

Bugfix: The boostrap script should not fail if documentation directory is empty.

Instead of failing, print a warning and continue onward.

Cleanup: Minor syntax mistakes in the documentation comments.

These were randomly discovered while I was working on the 0.6 branch man pages.

Bugfix: The build scripts are not handling documentation directory.

The man pages are not being correctly copied over, if at all.

Update: Add documentation comments regarding problems observed with gethostbyaddr() and gethostbyname().

Valgrind reported still reachable memory under a Glibc system when the network is not connected while performing a DNS resolution operation.

There is nothing I can do about this other than recommending to use something other than Glibc such as musl-libc.

Cleanup: Restructure time related types and add f_date_spec_t.

Make the time based types more interchangeable by using f_time_t.

Create a new time type called f_date_spec_t that utilizes f_time_spec_t instead of f_time_t for the time property.

Update: Relax the requirement in the license to be less verbose.

Do not specifically use my name "Kevin Day".
Instead refer to the "original license owner" or some authorized party.
This will make the license more flexible.

Feature: Add f_network_from_ip_address() and f_network_from_ip_name().

These are added to provide wrappers to gethostbyaddr() and gethostbyname().

Update the unit tests.

Bugfix: FSS Identify code accidentally got insert into the UTF8 program.

Probably an accidental mass find and replace somehow made its way into this file.

Update: Add unit tests for f_files_t and f_filess_t.

I forgot to add the unit tests with the commit fc1d33e993e99e8ff34d82be7fabadf7608d9c6d.

This also addresses a define name design problem where I have two cases that _F_type_array_file_h would be defined.
For now, just rename the _F_type_array_file_h to _F_type_array__file_h for the top-level file.

Cleanup: Remove unused files.

Feature: Add f_files_t and f_filess_t.

Provide arrays of f_file_t.

Bugfix: The level and monolithic build settings files are missing the poll source and header files.

Cleanup: Remove 'data_make.' from documentation comments in projects that do not use data_make.

Bugfix: Incorrect wrapping disable macro names for main functions.

The "_data_" should not be part of the macro name when it is not in the function name.

Update: Reduce NULL parameter requirements in f_file_select() regarding timeout.

The man pages state that the read, write, and except can all be NULL if timeout is not NULL.
Change the behavior of f_file_select() to allow this.

Update: The FSS Identify program to the latest practices.

Cleanup: Documentation comments, adding missing colon and fixing word usage.

Feature: Add f_file_select() for select().

This is more of a wrapper around select() (unlike f_file_poll() which is a bit more than a wrapper around poll()).
The arguments would be more difficult than worth it to simplify and normalize in the FLL typedef style.

Cleanup: Add quotes to testfile LD_LIBRARY_PATH assignment.

Do this for consistency with the pattern of the rest of the testfile.

Feature: Add additional network and socket related functions.

Add the following:
- f_file_poll() for poll().
- f_socket_read_stream() for recv().
- f_socket_write_stream() for send().

Add f_poll_t, f_polls_t, and f_pollss_t typedefs.
This adds the appropriate array management functions.
To be consistent with type_file.h a new type_array_file.h is created to manage these.

Update the existing f_signal_read() to use the new typdef f_poll_t.

This includes unit tests as appropriate.

Cleanup: Add environment to all unit test settings files.

Feature: Add f_network_to_ip_string() and f_network_from_ip_string() functions.

Provide support for inet_ntop() and inet_pton().

String are handling in FLL via f_string_static_t and f_string_dynamic_t.
The functions f_network_to_ip_string() and f_network_from_ip_string() are written to convert to these string types.

This implements a custom f_network_family_ip_t type to pass the digit IP address.
The idea here is to help reduce how much the FLL API is locked into the "libc" or POSIX API.

The structures are allocated using "{ 0 }" to take advantage of the supposed relatively new feature that treats this as an initialize all to NULL.

Cleanup: Fix documentation comment in f_memory.

Update: Remove unneeded variable storage and explicitly check for "-1" rather than "< 0".

Update: Rebuild the "disable" build files.

This needs to be done periodically when there are changes to functions.
Following the addition of f_network is a good point in time to do this.

Feature: Add f_network level 0 project, starting with host to/from network byte order functions.

As with most such level 0 projects these are just wrappers to the standard htonl() and similar functions.

This also updates the build settings and other related files.

The f_type_array also has some problems in the level and monolithic build settings fixed.

Cleanup: Fix incorrect comment in f_limit.

Feature: Add f_file_manipulate() as a wrapper for fcntl().

This also adds the unit tests.
The unit tests includes additional fixes where several of the wrapped functions are not being setup properly.
The fact that has not triggered an error in the past is concerning.

Feature: Add F_again and F_again_not status codes.

The fcntl() function uses two forms of privilege failures.
The EPERM is already translated to F_prohibited.
Therefore, add F_again (and F_again_not) so that EAGAIN can be used for that.

Feature: Add f_socket_name_host() and f_socket_name_peer().

Add missing network socket related functions for getting the "name" of the local or remote connections.

The local name is retuned as a string.

The remote name is returned on the f_socket_t structure.
I considered doing the work necessary to better generalize this but I decided not to.
I can leave that for once I am better experience in the network programming.

Cleanup: Rename *_total_parameters_d to *_parameter_total_d.

The naming scheme for the total parameters define is out of synchronization with the practices.
Rename this to be more in line with the current practices.

Cleanup: Add missing f_thread dependency.