Skip to content

Set RedisModule_StringCompare input args as const#10996

Merged
oranagra merged 1 commit intoredis:unstablefrom
gkorland:patch-4
Jul 17, 2022
Merged

Set RedisModule_StringCompare input args as const#10996
oranagra merged 1 commit intoredis:unstablefrom
gkorland:patch-4

Conversation

@gkorland
Copy link
Contributor

No description provided.

Copy link
Member

@oranagra oranagra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think this can be merged without concerns, following the footsteps and discussion in #10064 and #10733

@oranagra oranagra merged commit 587fc9c into redis:unstable Jul 17, 2022
@oranagra
Copy link
Member

ohh, only now i realized that you didn't also modify RM_StringCompare in module.c.
can you make another PR?

enjoy-binbin added a commit to enjoy-binbin/redis that referenced this pull request Jul 19, 2022
Following redis#10996, it forgot to modify RM_StringCompare in module.c

Modified RM_StringCompare, compareStringObjectsWithFlags,
compareStringObjects and collateStringObjects.
oranagra pushed a commit that referenced this pull request Jul 19, 2022
Following #10996, it forgot to modify RM_StringCompare in module.c

Modified RM_StringCompare, compareStringObjectsWithFlags,
compareStringObjects and collateStringObjects.
enjoy-binbin pushed a commit to enjoy-binbin/redis that referenced this pull request Jul 31, 2023
enjoy-binbin added a commit to enjoy-binbin/redis that referenced this pull request Jul 31, 2023
Following redis#10996, it forgot to modify RM_StringCompare in module.c

Modified RM_StringCompare, compareStringObjectsWithFlags,
compareStringObjects and collateStringObjects.
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