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.

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

Properly handle empty values #52

Merged
merged 1 commit into from
Jul 15, 2013
Merged

Properly handle empty values #52

merged 1 commit into from
Jul 15, 2013

Commits on Jul 10, 2013

  1. Allow empty values

    It is a legitimate use case to set empty values; to override a
    default when an empty value is acceptable for instance. This patch
    changes the regex in three ways: it 1) removes the requirement for
    a non-whitespace terminator on a setting value, 2) makes the value
    match non-greedy so that the \s*$ at the end can catch the newline
    and 3) changes the \s*=\s* to [ \t]*=[ \t]* because we don't want
    that to capture *any* whitespace (like a newline).
    otherwiseguy committed Jul 10, 2013
    Configuration menu
    Copy the full SHA
    5904c6f View commit details
    Browse the repository at this point in the history