Skip to content

Conversation

@rsarwad
Copy link
Owner

@rsarwad rsarwad commented Jan 6, 2023

Summary

fix(agw): Modified code to port spgw's state contexts to protobuf structure

Test Plan

Executed s1ap sanity test suite and unit test cases

@github-actions
Copy link

github-actions bot commented Jan 6, 2023

Thanks for opening a PR! 💯

A couple initial guidelines

Howto

  • Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
  • Checks. All required CI checks must pass before merge.
  • Merge. Once approved and passing CI checks, use the ready2merge label to indicate the maintainers can merge your PR.

More info

Please take a moment to read through the Magma project's

If this is your first Magma PR, also consider reading


free_wrapper((void**)&state_cache_p);
state_cache_p->Clear();
free_cpp_wrapper((void**)&state_cache_p);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[cpplint] reported by reviewdog 🐶
Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants