[Rpm-maint] [rpm-software-management/rpm] Warn if a text is after %else or %endif (#625)

Panu Matilainen notifications at github.com
Tue Feb 26 13:39:09 UTC 2019


pmatilai commented on this pull request.



> +    { 0, 0, 0, 0 }
+};
+
+static parsedSpecLine parseLineType(char *line)
+{
+    parsedSpecLine condition;
+
+    for (condition = implementedTypes; condition->text != NULL; condition++) {
+	if (condition->withArgs) {
+	    if (ISMACROWITHARG(line, condition->text, condition->textLen))
+		return condition;
+	} else {
+	    if (ISMACRO(line, condition->text, condition->textLen))
+		return condition;
+	}
+

Cosmetics: stray empty line

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/625#pullrequestreview-207957392
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190226/c68c5143/attachment.html>


More information about the Rpm-maint mailing list