-
Notifications
You must be signed in to change notification settings - Fork 592
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
fix(agw): Deleted temporary file emm_headers.hpp and moved functions out of extern #14489
fix(agw): Deleted temporary file emm_headers.hpp and moved functions out of extern #14489
Conversation
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
…ove_emm_headers_file
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.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
|
@pruthvihebbani, can you fix the conflicts? |
@@ -55,7 +55,6 @@ extern "C" { | |||
} | |||
#endif |
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.
Space
@@ -26,7 +26,6 @@ extern "C" { | |||
} | |||
#endif | |||
#include "lte/gateway/c/core/oai/tasks/nas/emm/emm_data.hpp" |
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.
Space
@@ -23,7 +23,6 @@ extern "C" { | |||
} | |||
|
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.
Remove dynamic_memory_check.h from extern C
@@ -34,7 +36,6 @@ extern "C" { | |||
#include "lte/gateway/c/core/oai/include/mme_config.hpp" | |||
#include "lte/gateway/c/core/oai/include/s6a_messages_types.hpp" | |||
#include "lte/gateway/c/core/oai/tasks/s6a/s6a_messages.hpp" | |||
#include "lte/gateway/c/core/oai/tasks/s6a/s6a_fd_iface.hpp" | |||
#include "orc8r/gateway/c/common/service303/MetricsHelpers.hpp" | |||
|
|||
#include <iostream> |
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.
Move these includes above
@@ -24,6 +24,7 @@ | |||
|
|||
#pragma once | |||
|
|||
#include "S1ap_S1AP-PDU.h" | |||
#include "lte/gateway/c/core/common/common_defs.h" | |||
#include "lte/gateway/c/core/oai/common/common_types.h" | |||
#include "lte/gateway/c/core/oai/include/mme_app_messages_types.hpp" |
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.
Include bstrlib.h, s1ap_messages_types.h under extern C
@@ -22,6 +22,8 @@ | |||
\email: lionel.gauthier@eurecom.fr | |||
*/ | |||
|
|||
#include "lte/gateway/c/core/oai/tasks/s1ap/s1ap_mme_nas_procedures.hpp" | |||
|
|||
#include <stdio.h> | |||
#include <stdint.h> | |||
#include <stdlib.h> |
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.
Move bstrlib.h file under extern C.
Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
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.
LGTM
…out of extern (magma#14489) * Deleted temporary file emm_headers.hpp and moved functions out of extern Signed-off-by: Pruthvi Hebbani <pruthvi.hebbani@radisys.com>
fix(agw): Deleted temporary file emm_headers.hpp and moved functions out of extern
Summary
In order to resolve undefined references, function declarations that were causing undefined references were commented out in the original files and added to a temporary file emm_headers.hpp. Now that all the files in mme_app and nas are migrated to C++, emm_headers.hpp is not needed. In this PR emm_headers.hpp is deleted and the function declarations are uncommented.
Test Plan
Executed the following builds/tests: