Skip to content

Modify parameter of clientMatchesFilter function.#1733

Merged
zuiderkwast merged 1 commit intovalkey-io:unstablefrom
RayaCoo:clientFilter-param-pointer
Feb 15, 2025
Merged

Modify parameter of clientMatchesFilter function.#1733
zuiderkwast merged 1 commit intovalkey-io:unstablefrom
RayaCoo:clientFilter-param-pointer

Conversation

@RayaCoo
Copy link

@RayaCoo RayaCoo commented Feb 14, 2025

small change to #1401.
pass clientFilter * to clientMatchesFilter as @zuiderkwast suggest in comment.

@codecov
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.95%. Comparing base (c5ae37f) to head (6634a15).
Report is 2 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #1733      +/-   ##
============================================
- Coverage     71.17%   70.95%   -0.22%     
============================================
  Files           123      123              
  Lines         65536    65536              
============================================
- Hits          46645    46503     -142     
- Misses        18891    19033     +142     
Files with missing lines Coverage Δ
src/networking.c 88.73% <100.00%> (-0.16%) ⬇️

... and 13 files with indirect coverage changes

@RayaCoo RayaCoo force-pushed the clientFilter-param-pointer branch from bda5d3c to 6634a15 Compare February 14, 2025 09:22
@zuiderkwast zuiderkwast merged commit cec9e5c into valkey-io:unstable Feb 15, 2025
50 checks passed
zuiderkwast pushed a commit that referenced this pull request Feb 15, 2025
small change to [#1401](#1401).
pass `clientFilter *` to clientMatchesFilter as suggest in
[comment](6bc64ca#r1875844273).

Signed-off-by: Ray Cao <[email protected]>
@RayaCoo RayaCoo deleted the clientFilter-param-pointer branch February 15, 2025 13:44
zuiderkwast pushed a commit that referenced this pull request Mar 18, 2025
small change to [#1401](#1401).
pass `clientFilter *` to clientMatchesFilter as suggest in
[comment](6bc64ca#r1875844273).

Signed-off-by: Ray Cao <[email protected]>
xbasel pushed a commit to xbasel/valkey that referenced this pull request Mar 27, 2025
small change to [valkey-io#1401](valkey-io#1401).
pass `clientFilter *` to clientMatchesFilter as suggest in
[comment](valkey-io@6bc64ca#r1875844273).

Signed-off-by: Ray Cao <[email protected]>
xbasel pushed a commit to xbasel/valkey that referenced this pull request Mar 27, 2025
small change to [valkey-io#1401](valkey-io#1401).
pass `clientFilter *` to clientMatchesFilter as suggest in
[comment](valkey-io@6bc64ca#r1875844273).

Signed-off-by: Ray Cao <[email protected]>
murphyjacob4 pushed a commit to enjoy-binbin/valkey that referenced this pull request Apr 13, 2025
small change to [valkey-io#1401](valkey-io#1401).
pass `clientFilter *` to clientMatchesFilter as suggest in
[comment](valkey-io@6bc64ca#r1875844273).

Signed-off-by: Ray Cao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants