Thanks for the patch.
Applied to 5.2, 5.3, 5.4 and trunk in revision 16868.
For the future, please send patches to the patch tracker.

On s=F6n, 2008-03-16 at 14:20 +0100, Magnus Fromreide wrote:
> On s=F6n, 2008-03-16 at 13:07 +0300, Aleksandr Lomanov wrote:
> > Hi all!
> > File read_config.c
> > When I calling register_prenetsnmp_mib_handler() with "type" parameter
> > equal to "snmp:snmpapp:..." it calls internal_register_config_handler()
> > with "when" parameter equal to PREMIB_CONFIG.
> > internal_register_config_handler() for processing "type_param" which
> > equal to "snmp:snmpapp:..." calls recursively
> > register_config_handler() which calls internal_register_config_handler()
> > with "when" parameter equal to NORMAL_CONFIG.

> =


> I do agree with you - this is a bug and it should be fixed. Now, the
> reason for the code to look as it does is backwards compatibility since
> the : had it even prior to the introduction of
> internal_register_config_handler.
> =


> The problem is how this should be handled when combined with the release
> policy[1] where API changes should be kept backwards-compatible if
> possible. This change wil change the API of
> register_prenetsnmp_mib_handler and while I doubt that anyone wil argue
> that the change is for the worse it still is a change and so I suppose
> there should be a new interface as opposed to the old one.
> =


> With all this said I think the patch should be applied anyhow.
> =


> /MF
> =


> [patch skipped]
> =


> 1) http://www.net-snmp.org/dev/release-policy.html
> =


> =


> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> Net-snmp-coders mailing list
> Net-snmp-coders@lists.sourceforge.net
> https://lists.sourceforge.net/lists/...et-snmp-coders



-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216...et/marketplace
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/...et-snmp-coders