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

Add DDS IDL Namespace Primitives and Normalize DdsDcpsCore.idl #3994

Merged
merged 3 commits into from
Jan 25, 2023

Conversation

iguessthislldo
Copy link
Member

These are defined in XTypes 1.3 Table 43. They are specified in the context of the C++ language binding, but it makes more sense for them to be defined in IDL.

These are defined in XTypes 1.3 Table 43. They are specified in the
context of the language binding, but makes more sense for them to be
defined in IDL.
dds/DdsDcpsCore.idl Outdated Show resolved Hide resolved
dds/DdsDcpsCore.idl Show resolved Hide resolved
Also:
  - Some cleanup in be_global that was going to come with some
    additional code to inject the `using namespace IDLv4;` like the
    includes are, but that ended up being unnecessary because of
    be_produce.
  - Fix problem compiling dds/CORBA/tao/Exception.cpp with TAO 3.
@mitza-oci
Copy link
Member

@mitza-oci mitza-oci merged commit 69c8d1f into OpenDDS:master Jan 25, 2023
iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this pull request Jan 25, 2023
OpenDDS#3994

- Fixed missing `TAO_VERSION_CODE` on TAO 2.2 by using
  `TAO_MAJOR_VERSION` instead.
- Replaced reference to bugzilla ticket with
  DOCGroup/ACE_TAO#2037
- Cleaned up and added sorting option for `update_gitignore.pl` used in
  OpenDDS#3994
- Replaced sprintf usage to satisfy build_m12_i0_j_FM-1f GHA warnings
mitza-oci added a commit that referenced this pull request Apr 26, 2024
Add `DDS` IDL Namespace Primitives and Normalize `DdsDcpsCore.idl`

(cherry picked from commit 69c8d1f)

# Conflicts:
#	dds/DdsDcpsCore.idl
#	dds/idl/be_produce.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants