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.

Sign up for GitHub

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

Jump to bottom

Remove unnecessary null pointer checks #6

Closed
elfring opened this issue May 12, 2016 · 3 comments
Closed

Remove unnecessary null pointer checks #6

elfring opened this issue May 12, 2016 · 3 comments
Labels
enhancement

Comments

@elfring
Copy link

elfring commented May 12, 2016

An extra null pointer check is not needed in functions like the following.
@baotiao
Copy link
Contributor

baotiao commented May 12, 2016

Yes, you are right. we don't need to check whether the point is NULL before we delete it.
can you submit an issue to help us fix these bugs?

@baotiao
Copy link
Contributor

baotiao commented May 17, 2016

we have removed these unnecessary pointers

@baotiao baotiao closed this as completed May 17, 2016
@elfring
Copy link
Author

elfring commented May 17, 2016

Thanks for your small source code improvement.

@wyf0705 wyf0705 mentioned this issue May 4, 2017
Closed
AlexStocks pushed a commit to ipixiu/pika that referenced this issue Dec 21, 2018
@Leviathan1995 @KernelMaker
bugfix Expire() and Makefile (OpenAtomFoundation#6)
63e292f 
* update Makefile

* bugfix Expire() and Makefile

* fix conflict

* fix conflict

* fix conlict

* fix conlict

* add packages in travis

* modify Expire()

* fix Del()

* add comments

* compatiable with redis protocol

* optimize code
AlexStocks pushed a commit to ipixiu/pika that referenced this issue Dec 21, 2018
@git-hulk @Axlgrep
MOD: clean-tidy the GetKeyNum and GetProperty in blackwidow.cc (OpenA…
db19eab 
…tomFoundation#6)

* MOD: clean-tidy the GetKeyNum and GetProperty in blackwidow.cc

* Fix: the iterator the order of db in GetKeyNum
@jencoldeng jencoldeng mentioned this issue Feb 12, 2019
Closed
@hotrhino hotrhino mentioned this issue Jul 18, 2019
Closed
@koolhazz koolhazz mentioned this issue May 9, 2020
Closed
luky116 referenced this issue in luky116/pika May 22, 2023
@luky116
Merge pull request #6 from luky116/newCode2
a964610 
fix: remove new/delete in net (OpenAtomFoundation#1503)
AlexStocks pushed a commit to AlexStocks/pika that referenced this issue May 26, 2023
@ForestLH
Merge pull request OpenAtomFoundation#6 from ForestLH/style/clang
cb7238a 
style:format codes
@bigdaronlee163 bigdaronlee163 mentioned this issue May 24, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@baotiao @elfring @flabby