<p dir="ltr">I don't really like that both the derived and the ancestor class has provides the same method and they.need to be called separately.</p>
<p dir="ltr">I think the one provided by threaded-dest-driver should be renamed to something else.</p>
<div class="gmail_quote">On Oct 18, 2013 5:11 PM, "Gergely Nagy" <<a href="mailto:algernon@balabit.hu">algernon@balabit.hu</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Gergely Nagy <<a href="mailto:algernon@balabit.hu">algernon@balabit.hu</a>> writes:<br>
<br>
> No, it's not the correct fix as it turns out, as init_method() will<br>
> override functions set earlier, if moved to later. So init_method()<br>
> needs to be a little smarter too.<br>
<br>
I ended up doing something else: I split the<br>
log_dest_driver_init_method() and log_threaded_dest_driver_init_method()<br>
invocations, the modules do both now. They do the firt in the beginning,<br>
the other one at the end. The _start_driver() function was merged into<br>
log_threaded_dest_driver_init_method() too.<br>
<br>
All LogThrDestDriver users in syslog-ng itself and in the Incubator have<br>
been updated accordingly.<br>
<br>
--<br>
|8]<br>
<br>
______________________________________________________________________________<br>
Member info: <a href="https://lists.balabit.hu/mailman/listinfo/syslog-ng" target="_blank">https://lists.balabit.hu/mailman/listinfo/syslog-ng</a><br>
Documentation: <a href="http://www.balabit.com/support/documentation/?product=syslog-ng" target="_blank">http://www.balabit.com/support/documentation/?product=syslog-ng</a><br>
FAQ: <a href="http://www.balabit.com/wiki/syslog-ng-faq" target="_blank">http://www.balabit.com/wiki/syslog-ng-faq</a><br>
<br>
</blockquote></div>