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

Fix wrong size calculation [11963] #2049

Merged
merged 7 commits into from
Jul 7, 2021
Merged

Fix wrong size calculation [11963] #2049

merged 7 commits into from
Jul 7, 2021

Conversation

richiware
Copy link
Member

Dataloans uses more size that allocated by Payload

@richiware richiware changed the title Fix wrong size calculation Fix wrong size calculation [11963] Jul 6, 2021
@richiware richiware force-pushed the bugfix/11962 branch 3 times, most recently from 79ae326 to a6ae2a3 Compare July 6, 2021 13:57
Signed-off-by: Ricardo González <ricardo@richiware.dev>
Signed-off-by: Ricardo González <ricardo@richiware.dev>
test/performance/latency/LatencyTestTypes.cpp Outdated Show resolved Hide resolved
test/performance/latency/LatencyTestTypes.cpp Outdated Show resolved Hide resolved
test/performance/throughput/ThroughputTypes.cpp Outdated Show resolved Hide resolved
test/performance/throughput/ThroughputTypes.cpp Outdated Show resolved Hide resolved
@MiguelCompany
Copy link
Member

Uncrustify should also be fixed

richiware and others added 2 commits July 7, 2021 08:16
Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Copy link
Member

@MiguelCompany MiguelCompany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uncrustify

test/performance/latency/LatencyTestTypes.hpp Outdated Show resolved Hide resolved
test/performance/throughput/ThroughputTypes.hpp Outdated Show resolved Hide resolved
richiware and others added 3 commits July 7, 2021 09:18
Signed-off-by: Ricardo González <ricardo@richiware.dev>
Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: Ricardo González <ricardo@richiware.dev>
Copy link
Member

@MiguelCompany MiguelCompany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with greenish CI

@MiguelCompany MiguelCompany merged commit dc19fe3 into master Jul 7, 2021
@MiguelCompany MiguelCompany deleted the bugfix/11962 branch July 7, 2021 14:41
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.

2 participants