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

app_perl: Make sv2int_str() static #1020

Merged
merged 1 commit into from Mar 14, 2017
Merged

app_perl: Make sv2int_str() static #1020

merged 1 commit into from Mar 14, 2017

Conversation

jhcloos
Copy link
Contributor

@jhcloos jhcloos commented Mar 6, 2017

The function is only called from the xs file, and C11’s inline
semantics mean that gcc-5 and later miscompile this file when
no optimization is specified.

Signed-off-by: James Cloos cloos@jhcloos.com

@miconda
Copy link
Member

miconda commented Mar 9, 2017

Can you update the commit message to follow the recommendations from contributions guide?

The commit messages are used to generate the changelog and it is good to have a coherence there.

@jhcloos jhcloos changed the title Make app_perl’s sv2int_str() static app_perl: Make sv2int_str() static Mar 10, 2017
- The function is only called from the xs file, and C11’s inline
  semantics mean that gcc-5 and later miscompile this file when
  no optimization is specified.

Signed-off-by: James Cloos <cloos@jhcloos.com>
@jhcloos
Copy link
Contributor Author

jhcloos commented Mar 10, 2017

Re-submitting.

@jhcloos jhcloos reopened this Mar 10, 2017
@jhcloos
Copy link
Contributor Author

jhcloos commented Mar 10, 2017

Github didn’t bother to email me your comment; I just happened on it today.

The current tip of https://github.com/jhcloos/kamailio.git master has the updated commit. I can’t tell whether this web ui is doing the right thing....

@miconda miconda merged commit 770331d into kamailio:master Mar 14, 2017
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