-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
INTERNAL: Rename update_filter to eflag_update #328
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 완료
libmemcached/collection.cc
Outdated
memcached_return_t memcached_coll_update_filter_init(memcached_coll_update_filter_st *ptr, | ||
const unsigned char *fvalue, | ||
const size_t fvalue_length) | ||
memcached_return_t memcached_coll_eflag_update_init(memcached_coll_update_filter_st *ptr, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
첫째 인자의 타입도 변경해야 합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정되었습니다.
libmemcached/collection.cc
Outdated
memcached_return_t memcached_coll_update_filter_set_bitwise(memcached_coll_update_filter_st *ptr, | ||
const size_t fwhere, | ||
memcached_coll_bitwise_t bitwise_op) | ||
memcached_return_t memcached_coll_eflag_update_set_bitwise(memcached_coll_update_filter_st *ptr, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기도 첫째 인자 타입을 변경
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 완료
libmemcached/types.h
Outdated
@@ -92,6 +92,7 @@ typedef struct memcached_coll_query_st memcached_bop_query_st; | |||
typedef struct memcached_hexadecimal_st memcached_hexadecimal_st; | |||
typedef struct memcached_mkey_st memcached_mkey_st; | |||
typedef struct memcached_coll_eflag_filter_st memcached_coll_eflag_filter_st; | |||
typedef struct memcached_coll_update_filter_st memcached_coll_eflag_update_st; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아래 형태가 되도록 구조체 이름도 변경합시다.
typedef struct memcached_coll_eflag_update_st memcached_coll_eflag_update_st;
typedef struct memcached_coll_eflag_update_st memcached_coll_update_filter_st;
🔗 Related Issue
⌨️ What I did