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

use oc_endpoint_to_string64 to utilize stack #530

Merged
merged 2 commits into from
Sep 20, 2023
Merged

use oc_endpoint_to_string64 to utilize stack #530

merged 2 commits into from
Sep 20, 2023

Conversation

jkralik
Copy link
Member

@jkralik jkralik commented Sep 14, 2023 •
edited
Loading

Fixes #529

@ocf-conformance-test-tool
Copy link

🎉 Thank you for your code contribution! To guarantee the change/addition is conformant to the OCF Specification, we would like to ask you to execute OCF Conformance Testing of your change ☝️ when your work is ready to be reviewed.

ℹ️ To verify your latest change (78703f1), label this PR with OCF Conformance Testing.

⚠️ Label is removed with every code change.

@jkralik jkralik force-pushed the jkralik/feature/optimize-oc-endpoint-to-string branch 12 times, most recently from 4aa80fa to b52d603 Compare September 15, 2023 08:22
@jkralik jkralik marked this pull request as ready for review September 15, 2023 09:29
@jkralik jkralik added the OCF Conformance Testing OCF Conformance Testing required label Sep 15, 2023
Danielius1922
Danielius1922 reviewed Sep 18, 2023
View reviewed changes
include/oc_helpers.h Outdated Show resolved Hide resolved
include/oc_helpers.h Outdated Show resolved Hide resolved
api/oc_endpoint.c Outdated Show resolved Hide resolved
@jkralik jkralik force-pushed the jkralik/feature/optimize-oc-endpoint-to-string branch from b52d603 to 2e0682c Compare September 18, 2023 13:03
@ocf-conformance-test-tool ocf-conformance-test-tool bot removed the OCF Conformance Testing OCF Conformance Testing required label Sep 18, 2023
@jkralik jkralik force-pushed the jkralik/feature/optimize-oc-endpoint-to-string branch 2 times, most recently from 57a329d to d0a3fdc Compare September 19, 2023 14:43
@Danielius1922 Danielius1922 force-pushed the jkralik/feature/optimize-oc-endpoint-to-string branch 2 times, most recently from 3a9a2aa to c194431 Compare September 20, 2023 08:27
@Danielius1922 Danielius1922 force-pushed the jkralik/feature/optimize-oc-endpoint-to-string branch from c194431 to 652a617 Compare September 20, 2023 09:44
@sonarcloud
Copy link

sonarcloud bot commented Sep 20, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

82.1% 82.1% Coverage
0.0% 0.0% Duplication

@Danielius1922 Danielius1922 merged commit 35f679b into master Sep 20, 2023
119 checks passed
@Danielius1922 Danielius1922 deleted the jkralik/feature/optimize-oc-endpoint-to-string branch September 20, 2023 10:43
@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Danielius1922 Danielius1922 Danielius1922 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Optimize memory allocation based on heaptrack results
2 participants
@jkralik @Danielius1922