diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2005-07-28 22:27:02 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2005-07-28 22:27:02 +0000 |
| commit | 5d27bf20b482ca7d1bd206da49bd6495d783bc71 (patch) | |
| tree | c2c18f7f6569033a0ca48ac9f8596602e66b1e03 /src/backend/utils/adt | |
| parent | ef85f5fabc0626228f7f75cbc6d3a93626691ea4 (diff) | |
| download | postgresql-5d27bf20b482ca7d1bd206da49bd6495d783bc71.tar.gz postgresql-5d27bf20b482ca7d1bd206da49bd6495d783bc71.zip | |
Make use of new list primitives list_append_unique and list_concat_unique
where applicable.
Diffstat (limited to 'src/backend/utils/adt')
| -rw-r--r-- | src/backend/utils/adt/acl.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/backend/utils/adt/acl.c b/src/backend/utils/adt/acl.c index 48a24db1826..d51162e7eed 100644 --- a/src/backend/utils/adt/acl.c +++ b/src/backend/utils/adt/acl.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/acl.c,v 1.122 2005/07/26 16:38:27 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/acl.c,v 1.123 2005/07/28 22:27:02 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -2843,8 +2843,7 @@ has_privs_of_role(Oid member, Oid role) * graph, we must test for having already seen this role. * It is legal for instance to have both A->B and A->C->B. */ - if (!list_member_oid(roles_list, otherid)) - roles_list = lappend_oid(roles_list, otherid); + roles_list = list_append_unique_oid(roles_list, otherid); } ReleaseSysCacheList(memlist); } @@ -2931,8 +2930,7 @@ is_member_of_role(Oid member, Oid role) * graph, we must test for having already seen this role. * It is legal for instance to have both A->B and A->C->B. */ - if (!list_member_oid(roles_list, otherid)) - roles_list = lappend_oid(roles_list, otherid); + roles_list = list_append_unique_oid(roles_list, otherid); } ReleaseSysCacheList(memlist); } @@ -3024,8 +3022,7 @@ is_admin_of_role(Oid member, Oid role) break; } - if (!list_member_oid(roles_list, otherid)) - roles_list = lappend_oid(roles_list, otherid); + roles_list = list_append_unique_oid(roles_list, otherid); } ReleaseSysCacheList(memlist); if (result) |