Re: [libsmi] patch to enhance range unions in smidump generated python & perl

On Thu, 2006-07-13 at 00:09 +0200, Juergen Schoenwaelder wrote:

[...]

...

I might not have been very clear. What I meant is to do what you propose with the minor change that the legacy range element would be indicating the full range and not just the last range element.

I agree - that sounds good. (I did indeed misinterpret what you said earlier)

I have checked in the modified patch. Here is now what I get for DateAndTime (SNMPv2-TC) which is defined as (SIZE (8 | 11)):

    "DateAndTime" : {
        "basetype" : "OctetString",
        "status" : "current",
        "ranges" : [
        {
            "min" : "8",
            "max" : "8"
        },
        {
            "min" : "11",
            "max" : "11"
        },
        ],
        "range" : {
            "min" : "8",
            "max" : "11"
        },

Thanks for submitting the patch.

Thanks for accepting it!

The pysnmp maintainers have already accepted my patch for libsmi2pysnmp which utilizes this fix.

--randy