-
Notifications
You must be signed in to change notification settings - Fork 380
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
IDLv4 Explicitly-named Integer Types #840
Merged
Merged
Changes from 14 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
8f4fcb1
TAO_IDL: IDL4 stdint-like Integers
iguessthislldo c7a6292
Add new limits for new types to Basic_Types.h
iguessthislldo ce28b30
ast_constant.cpp: Move C++ specific code out of the FE
iguessthislldo d004d8d
Adding Files As is For Merge
iguessthislldo 9d2523a
Merge remote-tracking branch 'upstream/master' into igtd/stdint
iguessthislldo 7549f0b
tao_idl: Committing Parser as is
iguessthislldo 85a1e61
Merge remote-tracking branch 'upstream/master' into igtd/stdint
iguessthislldo a4c5b22
Added WIP explicit_ints Test
iguessthislldo 56a61b9
Fix Issues with explicit_ints Test
iguessthislldo e994776
CDR_Stream.h: Fix Arg Type
iguessthislldo 2afc6a2
Fix Warnings, Add explicit_ints to Test List
iguessthislldo 737b919
Partially Respond to Review of #840
iguessthislldo 2280b39
Respond to Review of #840
iguessthislldo 7471fc9
ast_expression.cpp: Add Include for sprintf
iguessthislldo 800cec4
s/Uint/UInt/g
iguessthislldo 65962ee
Isolate IDL4 Ints in TAO Unless IDL4 is Used
iguessthislldo 71f7ce0
Respond to Review of #840
iguessthislldo b489eef
explicit_ints: Fix Mixup
iguessthislldo 2ed7fe7
Support int8/uint8 Sequences and Arrays
iguessthislldo 70b28ba
Try to Fix Issues with int8/uint8
iguessthislldo 354b698
Fix Issue with No Inlining
iguessthislldo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
We should convert these to
constexpr
at some pointThere 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.
We should, but I'd like that to be later because I don't want to make this harder to backport than it already is.