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

Remove redundant initialization #769

Merged
merged 1 commit into from Jan 28, 2021
Merged

Remove redundant initialization #769

merged 1 commit into from Jan 28, 2021

Conversation

ZhongxuanWang
Copy link
Contributor

Remove redundant initialization. We don't need to initialize t1 and t2 to a value since below k and p are explicitly defined in for loop and there's no other implicit conditions that may cause an runtime error if t1 and t2 are not initialized

Remove redundant initialization
@ZhongxuanWang
Copy link
Contributor Author

Remove redundant initialization. We don't need to initialize t1 and t2 to a value since below k and p are explicitly defined in for loop and there's no other implicit conditions that may cause an runtime error if t1 and t2 are not initialized

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 22.436% when pulling a784665 on ZhongxuanWang:patch-1 into 3d3fea0 on sarxos:master.

@sarxos sarxos merged commit 749b9ab into sarxos:master Jan 28, 2021
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

3 participants