Skip to content

#3453 make global function pointers defined in redismodule.h as extern#3455

Closed
levyfan wants to merge 1 commit intoredis:unstablefrom
levyfan:module
Closed

#3453 make global function pointers defined in redismodule.h as extern#3455
levyfan wants to merge 1 commit intoredis:unstablefrom
levyfan:module

Conversation

@levyfan
Copy link

@levyfan levyfan commented Aug 13, 2016

Global variables defined in headers will cause duplicate symbols linkage error when this header is included in multiple source files. We should declare it by extern in header and define it in c source file.
http://stackoverflow.com/questions/1433204/how-do-i-use-extern-to-share-variables-between-source-files-in-c

@levyfan
Copy link
Author

levyfan commented Aug 13, 2016

@antirez @itamarhaber @dvirsky please have a review #3453

@oranagra
Copy link
Member

oranagra commented Aug 8, 2020

Closing this one in favor of #6900. if you can, please test the changes in that PR and acknowledge that they indeed solve the problem. and post feedback in #6900.

@oranagra oranagra closed this Aug 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants