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.

Sign up for GitHub

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

Jump to bottom

IDL optional support #400

Merged
merged 17 commits into from
Sep 10, 2024
Merged

IDL optional support #400

merged 17 commits into from
Sep 10, 2024

Conversation

jwillemsen
Copy link
Member

No description provided.

@jwillemsen
Add optional CDR test
fc85a8d 
    * tests/idl4/optional/foo.cpp:
    * tests/idl4/optional/foo.h:
    * tests/idl4/optional/server.cpp:
      Added.

    * tests/idl4/optional/client.cpp:
    * tests/idl4/optional/test.idl:
    * tests/idl4/optional/test.mpc:
@jwillemsen
Further extend optional test IDL
6ebe064 
    * tests/idl4/optional/test.idl:
@jwillemsen
Extend optional support
cd7791c 
    * ridlbe/c++11/templates/cli/hdr/union_idl_traits_def.erb:
    * ridlbe/c++11/templates/cli/inl/except_inl.erb:
    * ridlbe/c++11/templates/cli/inl/union_inl.erb:
    * ridlbe/c++11/templates/cli/prx/union_cdr.erb:
    * ridlbe/c++11/visitors/union.rb:
    * tests/idl4/optional/test.idl:
@jwillemsen
member name duplicated
06c26f4 
    * ridlbe/c++11/templates/cli/hdr/union_idl_traits_def.erb:
@jwillemsen
Test extension
cd6c24a 
    * tests/idl4/optional/client.cpp:
    * tests/idl4/optional/test.idl:
@jwillemsen
Test fixes
9690dce 
    * ridlbe/c++11/templates/cli/inl/except_inl.erb:
    * ridlbe/c++11/templates/cli/inl/union_inl.erb:
    * ridlbe/c++11/visitors/struct.rb:
    * ridlbe/c++11/visitors/union.rb:
@jwillemsen
Further optimize and implement optional support
2151a1f 
    * ridlbe/c++11/templates/cli/prx/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * tao/x11/optional_cdr_t.h:
    * tests/idl4/optional/test.idl:
@jwillemsen
More optional work
598e892 
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * ridlbe/c++11/visitorbase.rb:
@jwillemsen
Reworked optional support
9cf7618 
    * ridlbe/c++11/config/cxx_type.rb:
    * ridlbe/c++11/templates/cli/hdr/union_idl_traits_def.erb:
    * ridlbe/c++11/templates/cli/prx/union_cdr.erb:
    * ridlbe/c++11/templates/cli/src/except_cdr.erb:
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/union_cdr.erb:
    * tao/x11/optional_cdr_t.h:
    * tests/idl4/optional/test.idl:
@jwillemsen
Working on runtime std::optional
fd9a7ca 
    * tests/idl4/default_literal/client.cpp:
    * tests/idl4/default_literal/run_test.pl:
    * tests/idl4/default_literal/test.idl:
    * tests/idl4/default_literal/test.mpc:
      Added.

    * bin/taox11_tests.lst:
    * ridlbe/c++11/config/cxx_type.rb:
    * ridlbe/c++11/templates/cli/hdr/union_idl_traits_def.erb:
    * ridlbe/c++11/templates/cli/prx/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/bitmask_cdr.erb:
    * ridlbe/c++11/templates/cli/src/bitset_cdr.erb:
    * ridlbe/c++11/templates/cli/src/enum_cdr.erb:
    * ridlbe/c++11/templates/cli/src/except_cdr.erb:
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/union_cdr.erb:
    * ridlbe/c++11/visitorbase.rb:
    * ridlbe/c++11/visitors/struct.rb:
    * ridlbe/c++11/visitors/union.rb:
    * tao/x11/optional_cdr_t.h:
    * tests/idl4/optional/client.cpp:
    * tests/idl4/optional/foo.cpp:
    * tests/idl4/optional/run_test.pl:
    * tests/idl4/optional/test.idl:
@jwillemsen
Runtime suppor
8c9950b 
    * tao/x11/optional_cdr_t.h:
@jwillemsen
Working CDR support
faecc1b 
    * ridlbe/c++11/config/cxx_type.rb:
    * ridlbe/c++11/templates/cli/src/except_cdr.erb:
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/union_cdr.erb:
    * tao/x11/optional_cdr_t.h:
@jwillemsen
Add test
dc594f4 
    * tests/idl4/optional/client.cpp:
    * tests/idl4/optional/foo.cpp:
    * tests/idl4/optional/test.idl:
@jwillemsen
ostream simplification
4aacb31 
    * tao/x11/base/idl_traits_t.h:
@jwillemsen
Removed not used methods
6bf09d4 
    * ridlbe/c++11/visitorbase.rb:
@jwillemsen
Simplified ostream insertion
8f82fa5 
    * ridlbe/c++11/config/cxx_type.rb:
    * ridlbe/c++11/templates/cli/hdr/struct_idl_traits_def.erb:
    * ridlbe/c++11/templates/cli/hdr/union_idl_traits_def.erb:
    * ridlbe/c++11/templates/cli/src/except_cdr.erb:
    * ridlbe/c++11/templates/cli/src/struct_cdr.erb:
    * ridlbe/c++11/templates/cli/src/union_cdr.erb:
    * ridlbe/c++11/visitorbase.rb:
    * tao/x11/optional_cdr_t.h:
    * tao/x11/optional_t.h:
@jwillemsen
Merge branch 'master' into jwi-optionaltest2
e83a505
@codecov Codecov
Copy link

codecov bot commented Sep 9, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 60.19417% with 41 lines in your changes missing coverage. Please review.

Project coverage is 59.80%. Comparing base (c0945bd) to head (e83a505).
Report is 18 commits behind head on master.

Files with missing lines Patch % Lines
tests/idl4/optional/server.cpp 55.31% 17 Missing and 4 partials ⚠️
tests/idl4/optional/client.cpp 51.21% 19 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #400      +/-   ##
==========================================
- Coverage   59.80%   59.80%   -0.01%     
==========================================
  Files         598      600       +2     
  Lines       47601    47697      +96     
  Branches     4216     4221       +5     
==========================================
+ Hits        28469    28524      +55     
- Misses      16010    16046      +36     
- Partials     3122     3127       +5

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jwillemsen jwillemsen merged commit 2845dd7 into RemedyIT:master Sep 10, 2024
14 checks passed
@jwillemsen jwillemsen deleted the jwi-optionaltest2 branch September 10, 2024 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jwillemsen