Summary:activeExpireCycleTryExpire use dbDelete

[xiaozhitaba]

BranchName: 6.0
CAFID: 0
BUGID: 0
WorkPackageName: redis
The ratio of self research code(0~1): 1

[xiaozhitaba]

why not use dbDelete in activeExpireCycleTryExpire function

[xiaozhitaba]

TCP_USER_TIMEOUT is needed

[xiaozhitaba]

SOCK_CLOEXEC need include <linux/tcp.h>

[xiaozhitaba]

CI / build-macos-latest (pull_request) Failing after 45s
MAKE hiredis
cd hiredis && /Applications/Xcode_11.3.1.app/Contents/Developer/usr/bin/make static
cc -std=c99 -pedantic -c -O3 -fPIC -I/usr/local/opt/openssl/include -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb net.c
net.c:44:10: fatal error: 'linux/tcp.h' file not found
#include <linux/tcp.h>

in my environment is ok

MAKE hiredis
cd hiredis && make static
make[3]: Entering directory '/root/redis/deps/hiredis'
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb net.c
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb hiredis.c
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sds.c
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb async.c
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb read.c
cc -std=c99 -pedantic -c -O3 -fPIC -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sockcompat.c
ar rcs libhiredis.a net.o hiredis.o sds.o async.o read.o sockcompat.o
make[3]: Leaving directory '/root/redis/deps/hiredis'

why @antirez @antirez

[xiaozhitaba]

I know macos does not have linux/tcp.h
but how to resolve this ？ @ @antirez please help me

[antirez]

Hello @xiaozhitaba! Too many changes in the same PR ;-D The dbDelete() thing is ok however. Please could you submit different changes in different PRs? Thanks.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}