Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] Remove redundant statement and return #481

Merged

Conversation

arthurkiller
Copy link
Contributor

  • freeClientArgv was previously defined in server.h
  • remove the redundant return

Signed-off-by: arthur.lee <arthur-lee@qq.com>
Copy link

codecov bot commented May 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.88%. Comparing base (fdd023f) to head (0bd85fe).

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable     #481      +/-   ##
============================================
+ Coverage     69.81%   69.88%   +0.06%     
============================================
  Files           109      109              
  Lines         61792    61791       -1     
============================================
+ Hits          43139    43180      +41     
+ Misses        18653    18611      -42     
Files Coverage Δ
src/aof.c 80.78% <ø> (-0.09%) ⬇️

... and 14 files with indirect coverage changes

@madolson madolson merged commit 2559f64 into valkey-io:unstable May 9, 2024
17 checks passed
karthyuom pushed a commit to karthyuom/valkey that referenced this pull request May 13, 2024
* `freeClientArgv` was previously defined in `server.h`
* remove the redundant return

Signed-off-by: arthur.lee <arthur-lee@qq.com>
Signed-off-by: Karthick Ariyaratnam <karthyuom@gmail.com>
srgsanky pushed a commit to srgsanky/valkey that referenced this pull request May 19, 2024
* `freeClientArgv` was previously defined in `server.h`
* remove the redundant return

Signed-off-by: arthur.lee <arthur-lee@qq.com>
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.

None yet

4 participants