Closed aquaticus closed 1 year ago
Thanks A lot for this contribution, it looks great. Just going to try and get the CI tests to run on this branch before it is merged, to make sure the tests pass. I'm going to close & re-open this to try to trigger the CI workflow to run.
Need to fix the CI tests, but I ran locally and things look good. Thanks again for these changes, appreciated!
const char* oid
instead ofchar* oid
as argumentconst string&
instead ofstring
addDynamicIntegerHandler()
addDynamicReadOnlyTimestampHandler()
addDynamicReadOnlyStringHandler()
int
(not onlyint*
)addGugeHandler()
marked as depreciated (new functionaddGaugeHandler()
).