Minor refactor in catcache.c
| От | cca5507 |
|---|---|
| Тема | Minor refactor in catcache.c |
| Дата | |
| Msg-id | tencent_12D586A7D4CB04E1A11C08D9E187FE15760A@qq.com обсуждение исходный текст |
| Ответы |
Re: Minor refactor in catcache.c
|
| Список | pgsql-hackers |
Hi,
```
@@ -1010,7 +1010,7 @@ RehashCatCache(CatCache *cp)
dlist_foreach_modify(iter, &cp->cc_bucket[i])
{
CatCTup *ct = dlist_container(CatCTup, cache_elem, iter.cur);
- int hashIndex = HASH_INDEX(ct->hash_value, newnbuckets);
+ Index hashIndex = HASH_INDEX(ct->hash_value, newnbuckets);
dlist_delete(iter.cur);
dlist_push_head(&newbucket[hashIndex], &ct->cache_elem);
@@ -1048,7 +1048,7 @@ RehashCatCacheLists(CatCache *cp)
dlist_foreach_modify(iter, &cp->cc_lbucket[i])
{
CatCList *cl = dlist_container(CatCList, cache_elem, iter.cur);
- int hashIndex = HASH_INDEX(cl->hash_value, newnbuckets);
+ Index hashIndex = HASH_INDEX(cl->hash_value, newnbuckets);
dlist_delete(iter.cur);
dlist_push_head(&newbucket[hashIndex], &cl->cache_elem);
```
The 'hashIndex' should be 'Index' rather than 'int'.
```
@@ -2039,8 +2039,7 @@ SearchCatCacheList(CatCache *cache,
#ifndef CATCACHE_FORCE_RELEASE
ct->dead &&
#endif
- ct->refcount == 0 &&
- (ct->c_list == NULL || ct->c_list->refcount == 0))
+ ct->refcount == 0)
CatCacheRemoveCTup(cache, ct);
}
```
Remove the dead code because we have a 'Assert(ct->c_list == NULL);'.
--
Regards,
ChangAo Chen
Вложения
В списке pgsql-hackers по дате отправления: