-
Notifications
You must be signed in to change notification settings - Fork 292
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
dlt-adaptor-udp, dlt-adaptor-stdin: implement get of verbosity level from input #9
Conversation
LocutusOfBorg
commented
Mar 10, 2017
src/adaptor/dlt-adaptor-stdin.c
Outdated
break; | ||
} else | ||
{ | ||
printf("Wrong verbosity level, setting default to INFO. Accepted values are: FATAL ERROR WARN INFO DEBUG VERBOSE\n"); |
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.
The default is already INFO. It is not being changed, so instead of "setting default to INFO" this should probably say: "setting to INFO" or "setting to default value (INFO)" or something similar.
src/adaptor/dlt-adaptor-udp.c
Outdated
break; | ||
} else | ||
{ | ||
printf("Wrong verbosity level, setting default to INFO. Accepted values are: FATAL ERROR WARN INFO DEBUG VERBOSE\n"); |
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.
See comment above
Sorry, I meant only to comment but clicked the wrong thing. I did not mean to Request Changes, or to Approve because it is the maintainers job to Approve, but I do not know how to remove "Request Changes" so I change it to "Approve" now. Added a minor comment. Otherwise +1 from me |
I did the above changes! thanks! let me know if I can merge the two commits together |
…from input, defaulting to INFO when wrong type or none is provided Signed-off-by: Gianfranco Costamagna <[email protected]>
61e24a7
to
4c757c0
Compare