-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fixed another batch of compilation warnings. #79
Conversation
warning: 'log_messages' defined but not used [-Wunused-variable]
warning: 'void do_log_syslog(const char*)' defined but not used [-Wunused-function]
looks good, but I propose to merge it after SLE12 as it do not bring nothing interesting in functionality |
I understand that you hesitate to merge this into RC2 before I disappear into the woods. But skipping SLE is too much; when I return I will convince you it is safe :) |
@jreidinger can we resolve this one? It takes only 10 minutes. I am willing to retract objectionable commits but would like to keep the obvious ones. |
warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
It was a TODO that went unfixed for 10+ years. Give way to real comipler warnings.
The method has been used in the YCP parser for the type of the container[indices]:default construct, so we are pretty sure it was in fact finished and complete, for al practical purposes. Give way to real compiler warnings.
|
OK, LGTM |
Fixed another batch of compilation warnings.
I tried to do only the "obvious" warnings. See commit messages for explanations.