@ffesti commented on this pull request.


> +         return p->token;
+       }
+    }
+    return NULL;
+}
+
+static const int partBeforeBuildOnly(int part)
+{
+    const struct PartRec *p;
+
+    for (p = partList; p->token != NULL; p++) {
+       if (p->part == part) {
+           return p->beforebuildonly;
+       }
+    }
+    return 0;

As I wrote above this is not quite as simple as it looks and I ma not that 
interested in refactoring the whole parting code. Yes, it looks wasteful and 
over complicated but I'd rather not change  a whole bunch of functions just to 
pass a new pointer around.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2917#discussion_r1514406101
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2917/review/1919642...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to