pmatilai commented on this pull request.


> @@ -2244,6 +2245,12 @@ int readManifest(rpmSpec spec, const char *path, const 
> char *descr, int flags,
            continue;
        if (specExpand(spec, lineno, buf, &expanded))
            goto exit;
+       if (flags & STRIP_TRAILINGSPACE) {
+           char *end = expanded + strlen(expanded) - 1;
+           while (end >= expanded && isspace((unsigned char)*end))

Use risspace() instead, it'll also save you from having to include ctype.h

-- 
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/1043#pullrequestreview-352813337
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to