-
Notifications
You must be signed in to change notification settings - Fork 15
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
Wrapper to send event messages is added #42
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #42 +/- ##
=======================================
Coverage 62.44% 62.44%
=======================================
Files 3 3
Lines 639 639
Branches 131 131
=======================================
Hits 399 399
Misses 165 165
Partials 75 75 Continue to review full report at Codecov.
|
ab2b149
to
7a8981e
Compare
parodusNotifier/main.c
Outdated
break; | ||
|
||
default: | ||
NotifError("Enter Valid commands..\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.
Put usage print to enter the required arguments
parodusNotifier/parodus_notifier.c
Outdated
.client_url = NULL | ||
}; | ||
|
||
ret = libparodus_init (&client_instance, &cfg); |
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.
This can be put under static variable so it executes only one time and instance can also be made static.
a802dea
to
906bbf7
Compare
906bbf7
to
61822be
Compare
Codecov Report
@@ Coverage Diff @@
## master #42 +/- ##
=======================================
Coverage 62.44% 62.44%
=======================================
Files 3 3
Lines 639 639
Branches 131 131
=======================================
Hits 399 399
Misses 165 165
Partials 75 75 Continue to review full report at Codecov.
|
No description provided.