Bug 2348 – patch: attribute for set of Ipv4Address instances

Bug 2348 - patch: attribute for set of Ipv4Address instances


Summary:	patch: attribute for set of Ipv4Address instances

Status:	PATCH PENDING

Product:	ns-3
Classification:	Unclassified
Component:	network
Version:	unspecified
Hardware:	All All

Importance:	P5 enhancement
Assigned To:	ns-bugs

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-03-25 17:27 EDT by Tom Henderson
Modified:	2016-03-26 12:26 EDT (History)
CC List:	1 user (show)

See Also:

Attachments
possible patch (6.37 KB, patch) 2016-03-25 17:28 EDT, Tom Henderson	Details \| Diff
possible patch (with IPv6) (9.22 KB, patch) 2016-03-26 12:26 EDT, Tom Henderson	Details \| Diff
Show Obsolete (1) Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Tom Henderson 2016-03-25 17:27:26 EDT

Possible implementation to respond to the request to be able to pass in a set of IP addresses as an attribute.  Thread is here.

http://mailman.isi.edu/pipermail/ns-developers/2016-March/013494.html

Comment 1 Tom Henderson 2016-03-25 17:28:59 EDT

Created attachment 2353 [details]
possible patch

This could of course be adapted for other address types.

Comment 2 Tommaso Pecorella 2016-03-26 08:42:06 EDT

(In reply to Tom Henderson from comment #1)
> Created attachment 2353 [details]
> possible patch
> 
> This could of course be adapted for other address types.

I see no problems at all in this attribute extension, but I have a question.
What about having an Address vector attribute (i.e., the base instance) ?

I haven't checked it, perhaps having just an Address vector will be a nightmare with the checker.

Comment 3 Tom Henderson 2016-03-26 12:26:15 EDT

Created attachment 2354 [details]
possible patch (with IPv6)

I will look into making this use base class Address; in the meantime, here is the v6-compatible version of the previous patch.