Skip to content

Commit f44f5da

Browse files
Update to newer form of adding class constants
This is a backport into cluster from the great pull request from @hylent
1 parent 4b974d2 commit f44f5da

File tree

1 file changed

+22
-22
lines changed

1 file changed

+22
-22
lines changed

redis.c

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -494,45 +494,45 @@ PHP_REDIS_API RedisSock *redis_sock_get_connected(INTERNAL_FUNCTION_PARAMETERS)
494494
/* Redis and RedisCluster objects share serialization/prefixing settings so
495495
* this is a generic function to add class constants to either */
496496
static void add_class_constants(zend_class_entry *ce, int is_cluster TSRMLS_DC) {
497-
add_constant_long(ce, "REDIS_NOT_FOUND", REDIS_NOT_FOUND);
498-
add_constant_long(ce, "REDIS_STRING", REDIS_STRING);
499-
add_constant_long(ce, "REDIS_SET", REDIS_SET);
500-
add_constant_long(ce, "REDIS_LIST", REDIS_LIST);
501-
add_constant_long(ce, "REDIS_ZSET", REDIS_ZSET);
502-
add_constant_long(ce, "REDIS_HASH", REDIS_HASH);
497+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_NOT_FOUND"), REDIS_NOT_FOUND TSRMLS_CC);
498+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_STRING"), REDIS_STRING TSRMLS_CC);
499+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_SET"), REDIS_SET TSRMLS_CC);
500+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_LIST"), REDIS_LIST TSRMLS_CC);
501+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_ZSET"), REDIS_ZSET TSRMLS_CC);
502+
zend_declare_class_constant_long(ce, ZEND_STRL("REDIS_HASH"), REDIS_HASH TSRMLS_CC);
503503

504504
/* Cluster doesn't support pipelining at this time */
505505
if(!is_cluster) {
506-
add_constant_long(ce, "PIPELINE", PIPELINE);
506+
zend_declare_class_constant_long(ce, ZEND_STRL("PIPELINE"), PIPELINE TSRMLS_CC);
507507
}
508508

509509
/* Add common mode constants */
510-
add_constant_long(ce, "ATOMIC", ATOMIC);
511-
add_constant_long(ce, "MULTI", MULTI);
510+
zend_declare_class_constant_long(ce, ZEND_STRL("ATOMIC"), ATOMIC TSRMLS_CC);
511+
zend_declare_class_constant_long(ce, ZEND_STRL("MULTI"), MULTI TSRMLS_CC);
512512

513513
/* options */
514-
add_constant_long(ce, "OPT_SERIALIZER", REDIS_OPT_SERIALIZER);
515-
add_constant_long(ce, "OPT_PREFIX", REDIS_OPT_PREFIX);
516-
add_constant_long(ce, "OPT_READ_TIMEOUT", REDIS_OPT_READ_TIMEOUT);
514+
zend_declare_class_constant_long(ce, ZEND_STRL("OPT_SERIALIZER"), REDIS_OPT_SERIALIZER TSRMLS_CC);
515+
zend_declare_class_constant_long(ce, ZEND_STRL("OPT_PREFIX"), REDIS_OPT_PREFIX TSRMLS_CC);
516+
zend_declare_class_constant_long(ce, ZEND_STRL("OPT_READ_TIMEOUT"), REDIS_OPT_READ_TIMEOUT TSRMLS_CC);
517517

518518
/* serializer */
519-
add_constant_long(ce, "SERIALIZER_NONE", REDIS_SERIALIZER_NONE);
520-
add_constant_long(ce, "SERIALIZER_PHP", REDIS_SERIALIZER_PHP);
519+
zend_declare_class_constant_long(ce, ZEND_STRL("SERIALIZER_NONE"), REDIS_SERIALIZER_NONE TSRMLS_CC);
520+
zend_declare_class_constant_long(ce, ZEND_STRL("SERIALIZER_PHP"), REDIS_SERIALIZER_PHP TSRMLS_CC);
521521

522522
/* scan options*/
523-
add_constant_long(ce, "OPT_SCAN", REDIS_OPT_SCAN);
524-
add_constant_long(ce, "SCAN_RETRY", REDIS_SCAN_RETRY);
525-
add_constant_long(ce, "SCAN_NORETRY", REDIS_SCAN_NORETRY);
523+
zend_declare_class_constant_long(ce, ZEND_STRL("OPT_SCAN"), REDIS_OPT_SCAN TSRMLS_CC);
524+
zend_declare_class_constant_long(ce, ZEND_STRL("SCAN_RETRY"), REDIS_SCAN_RETRY TSRMLS_CC);
525+
zend_declare_class_constant_long(ce, ZEND_STRL("SCAN_NORETRY"), REDIS_SCAN_NORETRY TSRMLS_CC);
526526

527527
/* Cluster option to allow for slave failover */
528528
if (is_cluster) {
529-
add_constant_long(ce, "OPT_SLAVE_FAILOVER", REDIS_OPT_FAILOVER);
530-
add_constant_long(ce, "FAILOVER_NONE", REDIS_FAILOVER_NONE);
531-
add_constant_long(ce, "FAILOVER_ERROR", REDIS_FAILOVER_ERROR);
532-
add_constant_long(ce, "FAILOVER_DISTRIBUTE", REDIS_FAILOVER_DISTRIBUTE);
529+
zend_declare_class_constant_long(ce, ZEND_STRL("OPT_SLAVE_FAILOVER"), REDIS_OPT_FAILOVER TSRMLS_CC);
530+
zend_declare_class_constant_long(ce, ZEND_STRL("FAILOVER_NONE"), REDIS_FAILOVER_NONE TSRMLS_CC);
531+
zend_declare_class_constant_long(ce, ZEND_STRL("FAILOVER_ERROR"), REDIS_FAILOVER_ERROR TSRMLS_CC);
532+
zend_declare_class_constant_long(ce, ZEND_STRL("FAILOVER_DISTRIBUTE"), REDIS_FAILOVER_DISTRIBUTE TSRMLS_CC);
533533
}
534534
#ifdef HAVE_REDIS_IGBINARY
535-
add_constant_long(ce, "SERIALIZER_IGBINARY", REDIS_SERIALIZER_IGBINARY);
535+
zend_declare_class_constant_long(ce, ZEND_STRL("SERIALIZER_IGBINARY"), REDIS_SERIALIZER_IGBINARY TSRMLS_CC);
536536
#endif
537537

538538
zend_declare_class_constant_stringl(ce, "AFTER", 5, "after", 5 TSRMLS_CC);

0 commit comments

Comments
 (0)