* We check for the connection state when receiving replies
so that we don't try to free(NULL)
* We now close the redis connection if we fail to send logs, which
causes us to reconnect in tryResume
* Cleaned up the freeReply code in endTransaction and dropped the
replies array since it isn't actually used.
as we now use libfastjson, we do no longer need to check for "new"
features, as these are always present in libfastjson (things
like json_object_object_get_ex()).
closes https://github.com/rsyslog/rsyslog/issues/935
here shoud free subscribes , becase CHKiRet(parseSubscriptions(es_str2cstr(pvals[i].val.d.estr, NULL), &inst->subscriptions)) have no way to free
the memory which malloc by es_str2cstr
those starting at column 0 are temporary debug aids during development.
Some of them I intend to keep in for a version or two. Some of them
I tend to forget later. This is a cleanup of those. Most are removed,
some are promoted to be permanently there - now indicated by proper
indention.
We started with char * pointers, but used different types of pointers
over time. This lead to alignment warnings. In practice, I think this
should never cause any problems (at least there have been no reports
in the 7 or so years we do this), but it is not clean. The interface is
now cleaned up. We do this in a way that does not require modifications
to modules that just use string parameters. For those with message
parameters, have a look at e.g. mmutf8fix to see how easy the
required change is.
Remove the use sizeof(char) or sizeof(uchar) in calculations for memory
allocation or string length. There are no known platforms for which
sizeof(char) or sizeof(uchar) is not 1, and c99 has defined sizeof(char)
to be 1 (section 6.5.3.4 of c99).
The previous commit build, but make distcheck failed.
Note that I also tried various things locally like changing library
position in the option line and adding -L at other places. None
works.
Instead of requiring the user to set the proper CFLAGS before invoking
./configure, we add CFLAGS and LIBS values as determined by pkg-config
during configuration setup.
We also update the requirements to be glib-2.0 instead of just plain
glib.
