[Ns-bugs] [Bug 1352] Map container reimplementation for attribute system

[code@nsnam.ece.gatech.edu]

https://www.nsnam.org/bugzilla/show_bug.cgi?id=1352

--- Comment #4 from Jaume Nin <jaume.nin at cttc.cat> 2012-03-14 12:58:16 EDT ---

Hi Mathieu,

First of all, apologies for the inconveniences and the little dedication I'd
been able to put in this issue and the missing files on the last submitted.
Patch generation is a brand new topic for me and it seems there is a lot of
room for improvement here.

Back to the basics. The main motivation for this feature is that in LENA we
have several map containers that need to be made available through the
attribute system in order to access the traces. The containers need to be maps
since their keys are not consecutive. A sample path of what we need follows:
/NodeList/#nodeId/DeviceList/#deviceId/LteUeRrc/RadioBearerMap/#RNTI/LteRlc/TxPDU

#RNTI is simply a user ID unique to the LTE network, hence we have things like
the following:
/NodeList/0/DeviceList/0/LteUeRrc/RadioBearerMap/1/LteRlc/TxPDU
/NodeList/0/DeviceList/0/LteUeRrc/RadioBearerMap/2/LteRlc/TxPDU
/NodeList/1/DeviceList/0/LteUeRrc/RadioBearerMap/3/LteRlc/TxPDU
/NodeList/1/DeviceList/0/LteUeRrc/RadioBearerMap/4/LteRlc/TxPDU

The current implementation, translates the map to a vector, hence the keys are
missed and the paths generated are:
/NodeList/0/DeviceList/0/LteUeRrc/RadioBearerMap/0/LteRlc/TxPDU
/NodeList/0/DeviceList/0/LteUeRrc/RadioBearerMap/1/LteRlc/TxPDU
/NodeList/1/DeviceList/0/LteUeRrc/RadioBearerMap/0/LteRlc/TxPDU
/NodeList/1/DeviceList/0/LteUeRrc/RadioBearerMap/1/LteRlc/TxPDU

The original idea was to avoid touching already working stuff, hence the
duplicity. However, from the maintainer point of view, it makes much more sense
to have a single ObjectPtrMapValue for both vectors and maps and probably the
solution will me more seamless, just the API and the container.

If you agree with this direction I'll try to provide the patch (clean and with
just the right number of whitespaces)  and extend the testing to cover maps as
well within a couple of weeks.

Regards,
Jaume

-- 
Configure bugmail: https://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.