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

delegate: Refactored delegate module to be more secure #7

Merged
merged 1 commit into from
Nov 27, 2017
Merged

delegate: Refactored delegate module to be more secure #7

merged 1 commit into from
Nov 27, 2017

Conversation

RicoAntonioFelix
Copy link

Change

  • Refactored delegate module to be more secure by taking references which can never be null

Signed-off-by: "Rico Antonio Felix" <"[email protected]">

@codecov
Copy link

codecov bot commented Nov 23, 2017

Codecov Report

Merging #7 into dev will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev       #7   +/-   ##
=======================================
  Coverage   98.01%   98.01%           
=======================================
  Files         247      247           
  Lines       64232    64232           
  Branches     1082     1082           
=======================================
  Hits        62959    62959           
  Misses        543      543           
  Partials      730      730
Impacted Files Coverage Δ
bfsdk/tests/test_delegate.cpp 100% <100%> (ø) ⬆️
bfsdk/include/bfdelegate.h 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 439c9d6...fa8f2df. Read the comment docs.

…rences which can never be null

Signed-off-by: "Rico Antonio Felix" <"[email protected]">
@rianquinn rianquinn merged commit ba077c2 into rianquinn:dev Nov 27, 2017
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