-
Notifications
You must be signed in to change notification settings - Fork 611
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Lucas Amaral <lucaaamaral@gmail.com>
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
There was a problem hiding this 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)); |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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.
int IdentityRequestMsg::DecodeIdentityRequestMsg( | ||
IdentityRequestMsg* identity_request, uint8_t* buffer, uint32_t len) { | ||
int IdentityRequestMsg::DecodeIdentityRequestMsg(uint8_t* buffer, | ||
uint32_t len) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
There was a problem hiding this comment.
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.
There was a problem hiding this 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, |
There was a problem hiding this comment.
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, |
There was a problem hiding this comment.
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, |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
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>
7fd229f
to
d7e93ec
Compare
There was a problem hiding this 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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
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]
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. |
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.
Additional Information
Security Considerations