[Rpm-maint] [rpm-software-management/rpm] Rip the marker support for multiline %{expr:...} error messages (#828)

Panu Matilainen notifications at github.com
Mon Sep 9 11:49:35 UTC 2019

pmatilai commented on this pull request.

> @@ -110,6 +110,12 @@ typedef struct _parseState {
 static void exprErr(const struct _parseState *state, const char *msg,
 		    const char *p)
+    const char *s = state->str;
+    s = strchr(s,'\n');
+    if (s && (*(s+1) != '\0'))
+	p = NULL;

While "s" is perfectly acceptable variable name in a case like this, a more descriptive one would make the code that little bit more readable. Say, "nl" or even "newline", after which the whole thing reads almost as human language. *Almost* :)

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190909/92e26ddd/attachment.html>

More information about the Rpm-maint mailing list