Bug 461 – RandomWaypointMobilityModel broken

Bug 461 - RandomWaypointMobilityModel broken


Summary:	RandomWaypointMobilityModel broken

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	mobility models
Version:	ns-3-dev
Hardware:	All All

Importance:	P1 major
Assigned To:	Mathieu Lacage

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-01-07 12:38 UTC by Mauro Tortonesi
Modified:	2009-01-09 02:34 UTC (History)
CC List:	0 users

See Also:

Attachments
patch from mailing list (1.06 KB, patch) 2009-01-09 02:28 UTC, Mathieu Lacage	Details \| Diff
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 Mauro Tortonesi 2009-01-07 12:38:21 UTC

Dear NS3 developers,

my colleague Stefano Formaggi and I have found a couple of bugs in the
RandomWaypointMobilityModel code:

1) both the constructor and the DoSetPosition method forget to set m_event
2) the BeginWalk method forgets to call m_helper.Unpause ()

These bugs effectively break support for random waypoint mobility in ns3.

The patch in attachment fixes both problems. It applies on both ns-3.3 and current ns-3-dev trunk.


Mauro Tortonesi

Comment 1 Rajib Bhattacharjea 2009-01-07 15:43:32 UTC

Assigned to the maintainer for review and application.

Comment 2 Mathieu Lacage 2009-01-09 02:27:13 UTC

There is no attachement ?

Comment 3 Mathieu Lacage 2009-01-09 02:28:42 UTC

Created attachment 348 [details]
patch from mailing list

Comment 4 Mathieu Lacage 2009-01-09 02:34:45 UTC

thanks a lot ! 
changeset 752edcf2480b