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

refactor(amf): use 'this' instead of redundant object pointer in methods #15565

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lucaaamaral
Copy link

Signed-off-by: Lucas Amaral lucaaamaral@gmail.com

refactor(amf): use 'this' instead of redundant object pointer in methods

Summary

Removed redundant object pointers inside classes methods, replacing them with the this pointer.

Test Plan

Tests were made by running the command below and checking the tests results.

bazel build lte/gateway/c/core:all && bazel test lte/gateway/c/core/oai/test/amf:all
[...]
INFO: Build completed successfully, 510 total actions
//lte/gateway/c/core/oai/test/amf:amf_algorithm_selection_test           PASSED in 0.7s
//lte/gateway/c/core/oai/test/amf:amf_common_utils_test                  PASSED in 0.7s
//lte/gateway/c/core/oai/test/amf:amf_encode_decode_test                 PASSED in 0.7s
//lte/gateway/c/core/oai/test/amf:amf_map_test                           PASSED in 0.7s
//lte/gateway/c/core/oai/test/amf:amf_procedures_test                    PASSED in 0.8s
//lte/gateway/c/core/oai/test/amf:amf_stateless_test                     PASSED in 0.7s

Additional Information

  • This change is backwards-breaking

Security Considerations

Signed-off-by: Lucas Amaral <lucaaamaral@gmail.com>
@lucaaamaral lucaaamaral requested review from a team as code owners November 6, 2024 03:24
Copy link
Contributor

github-actions bot commented Nov 6, 2024

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

@github-actions github-actions bot added the component: agw Access gateway-related issue label Nov 6, 2024
@pull-request-size pull-request-size bot added the size/XXL Denotes a Pull Request that changes 1000+ lines. label Nov 6, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

cpplint

lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionEstablishmentAccept.cpp|169| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionEstablishmentReject.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionEstablishmentReject.cpp|68| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionEstablishmentRequest.cpp|190| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionEstablishmentRequest.cpp|200| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationCommand.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationCommand.cpp|110| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationCommandReject.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationCommandReject.cpp|74| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationComplete.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationComplete.cpp|68| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationReject.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionModificationReject.cpp|85| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseCommand.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseCommand.cpp|36| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseReject.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseReject.cpp|36| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseRequest.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GPDUSessionReleaseRequest.cpp|62| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationAccept.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationAccept.cpp|68| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationComplete.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationComplete.cpp|62| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationReject.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationReject.cpp|67| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationRequest.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GRegistrationRequest.cpp|147| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeCommand.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeCommand.cpp|85| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeComplete.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeComplete.cpp|62| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeReject.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeReject.cpp|67| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|192| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|202| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|212| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/SmfMessage.cpp|28| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/SmfMessage.cpp|30| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationFailureIE.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterAUTN.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterAUTN.cpp|39| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterRAND.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterRAND.cpp|39| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|42| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|50| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|51| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GEAPMessage.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GEAPMessage.cpp|46| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer2.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer2.cpp|42| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer3.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer3.cpp|45| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GIntegrityProtMaxDataRate.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GIntegrityProtMaxDataRate.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMMCause.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMMCause.cpp|44| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|36| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNSSAI.cpp|142| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|50| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|51| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|58| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|59| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|54| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|29| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|52| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|53| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|46| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|26| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|21| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|75| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|88| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|96| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|127| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|56| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|45| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|91| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|120| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|236| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|260| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|294| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|295| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|428| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|429| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|40| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|41| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|19| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|117| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|118| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]

if (blength(str) <= 0) return 0;
if (buffer == NULL) return TLV_BUFFER_NULL;
if (buflen < blength(str)) return TLV_BUFFER_TOO_SHORT;
memcpy((void*)buffer, (void*)str->data, blength(str));
Copy link
Contributor

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]

Copy link
Author

Choose a reason for hiding this comment

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

This suggestion is impossible to comply as this is C code and not C++. I believe the reviewdog might be misconfigured.

@@ -887,7 +894,7 @@ int nas5g_message_decode(const unsigned char* const buffer,

int nas5g_message_encode(unsigned char* buffer,
const amf_nas_message_t* const msg, uint32_t length,
void* security);
amf_security_context_t* security);
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

@lucaaamaral lucaaamaral Nov 14, 2024

Choose a reason for hiding this comment

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

No change in indentation has been made, only the adjustment in the type to avoid unnecessary cast and increase safeness of code.
The indentation is automatically placed in the devcontainer environment.

@@ -61,7 +61,7 @@ int _nas5g_message_plain_encode(unsigned char* buffer,

static int _nas5g_message_protected_encode(
unsigned char* buffer, const nas_message_security_protected_t* msg,
uint32_t length, void* security);
uint32_t length, amf_security_context_t* security);
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the adjustment in the type to avoid unnecessary cast and increase safeness of code.
The indentation is automatically placed in the devcontainer environment.

@@ -224,74 +224,75 @@
***************************************************************************/
int nas5g_message_encode(unsigned char* buffer,
const amf_nas_message_t* const msg, uint32_t length,
void* security) {
amf_security_context_t* amf_security_context) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the adjustment in the type to avoid unnecessary cast and increase safeness of code.
The indentation is automatically placed in the devcontainer environment.

lte/gateway/c/core/oai/tasks/amf/nas5g_message.cpp Outdated Show resolved Hide resolved
int IdentityRequestMsg::DecodeIdentityRequestMsg(
IdentityRequestMsg* identity_request, uint8_t* buffer, uint32_t len) {
int IdentityRequestMsg::DecodeIdentityRequestMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the removal of unnecessary field to avoid misuse of method and ease the comprehension of the code.
The indentation is automatically placed in the devcontainer environment.

int IdentityRequestMsg::EncodeIdentityRequestMsg(
IdentityRequestMsg* identity_request, uint8_t* buffer, uint32_t len) {
int IdentityRequestMsg::EncodeIdentityRequestMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the removal of unnecessary field to avoid misuse of method and ease the comprehension of the code.
The indentation is automatically placed in the devcontainer environment.

int IdentityResponseMsg::DecodeIdentityResponseMsg(
IdentityResponseMsg* identity_response, uint8_t* buffer, uint32_t len) {
int IdentityResponseMsg::DecodeIdentityResponseMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the removal of unnecessary field to avoid misuse of method and ease the comprehension of the code.
The indentation is automatically placed in the devcontainer environment.

int IdentityResponseMsg::EncodeIdentityResponseMsg(
IdentityResponseMsg* identity_response, uint8_t* buffer, uint32_t len) {
int IdentityResponseMsg::EncodeIdentityResponseMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the removal of unnecessary field to avoid misuse of method and ease the comprehension of the code.
The indentation is automatically placed in the devcontainer environment.

@@ -27,8 +27,7 @@ PDUSessionEstablishmentAcceptMsg::~PDUSessionEstablishmentAcceptMsg(){};

// Decode PDUSessionEstablishmentAccept Message and its IEs
int PDUSessionEstablishmentAcceptMsg::DecodePDUSessionEstablishmentAcceptMsg(
PDUSessionEstablishmentAcceptMsg* pdu_session_estab_accept, uint8_t* buffer,
uint32_t len) {
uint8_t* buffer, uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

Copy link
Author

Choose a reason for hiding this comment

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

No change in indentation has been made, only the removal of unnecessary field to avoid misuse of method and ease the comprehension of the code.
The indentation is automatically placed in the devcontainer environment.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

cpplint

lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeCommand.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeCommand.cpp|85| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeComplete.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeComplete.cpp|62| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeReject.cpp|29| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GSecurityModeReject.cpp|67| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|192| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|202| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/M5GULNASTransport.cpp|212| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/SmfMessage.cpp|28| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/SmfMessage.cpp|30| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationFailureIE.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterAUTN.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterAUTN.cpp|39| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterRAND.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationParameterRAND.cpp|39| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|42| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|50| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GAuthenticationResponseParameter.cpp|51| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GEAPMessage.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GEAPMessage.cpp|46| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer2.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer2.cpp|42| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer3.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GGprsTimer3.cpp|45| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GIntegrityProtMaxDataRate.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GIntegrityProtMaxDataRate.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMMCause.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMMCause.cpp|44| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|36| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNSSAI.cpp|142| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|50| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|51| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|58| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|59| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|54| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|29| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|52| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|53| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|46| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|26| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|21| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|75| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|88| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|96| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|127| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|56| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|45| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|91| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|120| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|236| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|260| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|294| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|295| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|428| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|429| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|40| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|41| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|19| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|117| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|118| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]

const amf_nas_message_t* const msg, uint32_t length,
void* security) {
int nas5g_message_encode(unsigned char* buffer, const amf_nas_message_t* msg,
uint32_t length,
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

amf_msg_header* const header,
const uint32_t length,
amf_nas_message_decode_status_t* const status,
amf_msg_header* header, const uint32_t length,
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

const uint32_t length,
amf_nas_message_decode_status_t* const status,
amf_msg_header* header, const uint32_t length,
amf_nas_message_decode_status_t* status,
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

@@ -185,134 +166,112 @@ int PDUSessionEstablishmentAcceptMsg::DecodePDUSessionEstablishmentAcceptMsg(

// Encode PDUSessionEstablishmentAccept Message and its IEs
int PDUSessionEstablishmentAcceptMsg::EncodePDUSessionEstablishmentAcceptMsg(
PDUSessionEstablishmentAcceptMsg* pdu_session_estab_accept, uint8_t* buffer,
uint32_t len) {
uint8_t* buffer, uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

@@ -26,47 +26,37 @@ PDUSessionEstablishmentRejectMsg::~PDUSessionEstablishmentRejectMsg(){};

// Decode PDUSessionEstablishmentReject Message and its IEs
int PDUSessionEstablishmentRejectMsg::DecodePDUSessionEstablishmentRejectMsg(
PDUSessionEstablishmentRejectMsg* pdu_session_estab_reject, uint8_t* buffer,
uint32_t len) {
uint8_t* buffer, uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int RegistrationCompleteMsg::EncodeRegistrationCompleteMsg(
RegistrationCompleteMsg* reg_complete, uint8_t* buffer, uint32_t len) {
int RegistrationCompleteMsg::EncodeRegistrationCompleteMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int RegistrationRejectMsg::DecodeRegistrationRejectMsg(
RegistrationRejectMsg* reg_reject, uint8_t* buffer, uint32_t len) {
int RegistrationRejectMsg::DecodeRegistrationRejectMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int RegistrationRejectMsg::EncodeRegistrationRejectMsg(
RegistrationRejectMsg* reg_reject, uint8_t* buffer, uint32_t len) {
int RegistrationRejectMsg::EncodeRegistrationRejectMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int RegistrationRequestMsg::DecodeRegistrationRequestMsg(
RegistrationRequestMsg* reg_request, uint8_t* buffer, uint32_t len) {
int RegistrationRequestMsg::DecodeRegistrationRequestMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int RegistrationRequestMsg::EncodeRegistrationRequestMsg(
RegistrationRequestMsg* reg_request, uint8_t* buffer, uint32_t len) {
int RegistrationRequestMsg::EncodeRegistrationRequestMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

@lucaaamaral
Copy link
Author

All remaining points relates to indentation, I believe there are misconfiguration in the reviewdog or in the devcontainer indentation style.

Signed-off-by: Lucas Amaral <lucaaamaral@gmail.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

cpplint

lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMMCause.cpp|44| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GMaxNumOfSupportedPacketFilters.cpp|36| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNASSecurityAlgorithms.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNSSAI.cpp|142| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|50| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNasKeySetIdentifier.cpp|51| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|58| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GNetworkFeatureSupport.cpp|59| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUAddress.cpp|54| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|29| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|52| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionIdentity.cpp|53| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionReActivationResult.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPDUSessionType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainer.cpp|46| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|26| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GPayloadContainerType.cpp|35| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|21| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|75| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|88| Using C-style cast. Use static_cast<uint32_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|96| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GProtocolConfigurationOptions.cpp|127| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|30| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GQosFlowDescriptor.cpp|56| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSDeRegistrationType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|23| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSIdentityType.cpp|31| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMCause.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|45| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|91| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|120| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|236| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|260| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|294| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|295| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|428| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSMobileIdentity.cpp|429| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|40| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationResult.cpp|41| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|48| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSRegistrationType.cpp|49| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSecurityHeaderType.cpp|34| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|25| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GServiceType.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|19| You don't need a ; after a } [readability/braces] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GSpareHalfOctet.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|32| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|33| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|117| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUESecurityCapability.cpp|118| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|24| Do not indent within a namespace. [whitespace/indent_namespace] [4]
lte/gateway/c/core/oai/tasks/nas5g/src/ies/M5GUplinkDataStatus.cpp|37| Do not indent within a namespace. [whitespace/indent_namespace] [4]

int SecurityModeCommandMsg::DecodeSecurityModeCommandMsg(
SecurityModeCommandMsg* sec_mode_command, uint8_t* buffer, uint32_t len) {
int SecurityModeCommandMsg::DecodeSecurityModeCommandMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int SecurityModeCommandMsg::EncodeSecurityModeCommandMsg(
SecurityModeCommandMsg* sec_mode_command, uint8_t* buffer, uint32_t len) {
int SecurityModeCommandMsg::EncodeSecurityModeCommandMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int SecurityModeCompleteMsg::DecodeSecurityModeCompleteMsg(
SecurityModeCompleteMsg* sec_mode_complete, uint8_t* buffer, uint32_t len) {
int SecurityModeCompleteMsg::DecodeSecurityModeCompleteMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int SecurityModeCompleteMsg::EncodeSecurityModeCompleteMsg(
SecurityModeCompleteMsg* sec_mode_complete, uint8_t* buffer, uint32_t len) {
int SecurityModeCompleteMsg::EncodeSecurityModeCompleteMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int SecurityModeRejectMsg::DecodeSecurityModeRejectMsg(
SecurityModeRejectMsg* sec_mode_reject, uint8_t* buffer, uint32_t len) {
int SecurityModeRejectMsg::DecodeSecurityModeRejectMsg(uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int GPRSTimer3Msg::DecodeGPRSTimer3Msg(GPRSTimer3Msg* gprstimer, uint8_t iei,
uint8_t* buffer, uint32_t len) {
int GPRSTimer3Msg::DecodeGPRSTimer3Msg(uint8_t iei, uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int GPRSTimer3Msg::EncodeGPRSTimer3Msg(GPRSTimer3Msg* gprstimer, uint8_t iei,
uint8_t* buffer, uint32_t len) {
int GPRSTimer3Msg::EncodeGPRSTimer3Msg(uint8_t iei, uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

@@ -20,27 +20,21 @@ IntegrityProtMaxDataRateMsg::~IntegrityProtMaxDataRateMsg(){};

// Decode IntegrityProtMaxDataRate IE
int IntegrityProtMaxDataRateMsg::DecodeIntegrityProtMaxDataRateMsg(
IntegrityProtMaxDataRateMsg* integrity_prot_max_data_rate, uint8_t iei,
uint8_t* buffer, uint32_t len) {
uint8_t iei, uint8_t* buffer, uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

return (decoded);
};

// Encode IntegrityProtMaxDataRate IE
int IntegrityProtMaxDataRateMsg::EncodeIntegrityProtMaxDataRateMsg(
IntegrityProtMaxDataRateMsg* integrity_prot_max_data_rate, uint8_t iei,
uint8_t* buffer, uint32_t len) {
uint8_t iei, uint8_t* buffer, uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

int M5GMMCauseMsg::DecodeM5GMMCauseMsg(M5GMMCauseMsg* m5gmm_cause, uint8_t iei,
uint8_t* buffer, uint32_t len) {
int M5GMMCauseMsg::DecodeM5GMMCauseMsg(uint8_t iei, uint8_t* buffer,
uint32_t len) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[cpplint] reported by reviewdog 🐶
Do not indent within a namespace. [whitespace/indent_namespace] [4]

@lucaaamaral
Copy link
Author

Most of remaining points relates to indentation, I believe there are misconfiguration in the reviewdog or in the devcontainer indentation style. This makes it hard to fix the important points. I believe it needs a fix in those scripts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: agw Access gateway-related issue size/XXL Denotes a Pull Request that changes 1000+ lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant