Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Few parse_ini_config_file() fixes #127

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Few parse_ini_config_file() fixes #127

wants to merge 4 commits into from

Commits on Feb 27, 2024

  1. cqfd: comment alignment in parse_ini_config_file() 

    This fixes comment misalignment.
    @gportay
    gportay committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    3ed2912 View commit details
    Browse the repository at this point in the history

  2. cqfd: fix comment typo in parse_ini_config_file() 

    This fixes a typo in a comment; the variable expansion intends to remove
tabulation after the sign =.
    @gportay
    gportay committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    60d0c36 View commit details
    Browse the repository at this point in the history

  3. cqfd: fix tabulation character in variable expansions 

    The variable expansions intend to remove the tabulations before and
after the sign =, as per the comment.

However, the patterns match several spaces instead of a tabulation
(probably because of a copy/paste at code import).

This fixes the mismatch variable expansion by replacing the sequence of
spaces (backslash included) by a the single quoted string composed of
the ANSI escaped tabulation character (i.e. $'\t').
    @gportay
    gportay committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    d13611d View commit details
    Browse the repository at this point in the history

  4. cqfd: split space removal into two variable expansions 

    The removal of tabulation arround the sign = is perform by two variable
expansions:
 - remove tabulation before the sign =
 - remove tabulation after the sign =

This applies the same procedure to space by splitting the single removal
into two variable expansions: one before the sign =, one after the sign
=.
    @gportay
    gportay committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    1c2f9ab View commit details
    Browse the repository at this point in the history