| Message ID | dda81fedd9f38cb57658f1343f803a339c9116e3.1711001016.git.dsimic@manjaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 0d49b1e5a8c789dd3f1029f382bd76a464b72da0 |
| Headers | show |
| Series | Fix a bug in configuration parsing, and improve tests and documentation | expand |
diff --git a/config.c b/config.c index 3cfeb3d8bd99..a86a20cdf5cb 100644 --- a/config.c +++ b/config.c @@ -869,7 +869,7 @@ static char *parse_value(struct config_source *cs) continue; } if (c == '"') { - quote = 1-quote; + quote = 1 - quote; continue; } strbuf_addch(&cs->value, c);
In general, binary operators should be enclosed in a pair of leading and trailing space (SP) characters. Thus, clean up one spotted expression that for some reason had a "bunched up" operator. Signed-off-by: Dragan Simic <dsimic@manjaro.org> --- Notes: Changes in v5: - No changes were introduced Changes in v4: - Junio responded with "will queue" in v3, [1] but this patch hasn't reached the "next" branch yet, so resending it as a reminder - No changes were introduced Changes in v3: - Patch description was expanded a tiny bit, to make it more accurate - No changes to the source code were introduced Changes in v2: - No changes were introduced [1] https://lore.kernel.org/git/xmqq5xxhl8gs.fsf@gitster.g/ config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)