Added dependencies for arch-based systems in the README.md;
Removed the trim functions in the output.c, it was only used by print_battery_info (dead code);
Removed the forced trimming in print_battery_info;
Modifed the expected output of the 014 test. There is an space between
the two last %parameters, the expected output must reflect this space
character;
print_eth_info and print_wireless_info got modifed by running
clang-format -i */.[ch] (clang-format version 16.0.3) as specified
in the CONTRIBUTING.md file;
Note:
I believe the initial intent of the trimming was to remove spaces due
to empty values such as in this example: %remaining having no value
when the battery is at 100%.
It creates a tailing space, which was in the formatting, made by the user,
to separate the last two parameters.
format: ... %status %remaining
result: ... [status value][space][empty string here due to no value]
Although, even with the trimming, an empty value right in the middle of
a set of %parameters will still make empty spaces.
I tested the claim other modules not trimning and it was the true. In
other words, the trimming did not fix anything and it was causing this
module to behave abnormally.
496
Note: I believe the initial intent of the trimming was to remove spaces due to empty values such as in this example: %remaining having no value when the battery is at 100%.
It creates a tailing space, which was in the formatting, made by the user, to separate the last two parameters.
format: ... %status %remaining result: ... [status value][space][empty string here due to no value]
Although, even with the trimming, an empty value right in the middle of a set of %parameters will still make empty spaces.
I tested the claim other modules not trimning and it was the true. In other words, the trimming did not fix anything and it was causing this module to behave abnormally.