Bug 603 – Simulator::Next is useless

Bug 603 - Simulator::Next is useless


Summary:	Simulator::Next is useless

Status:	RESOLVED FIXED

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

Importance:	P5 normal
Assigned To:	ns-bugs

URL:
Keywords:

Depends on:	631
Blocks:
	Show dependency tree / graph

Reported:	2009-06-23 09:17 EDT by Mathieu Lacage
Modified:	2012-03-28 09:03 EDT (History)
CC List:	2 users (show)

See Also:

Attachments
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 Mathieu Lacage 2009-06-23 09:17:07 EDT

I would like to kill this method: no one uses it and I have spent a good amount of time searching my archives: no one ever used it outside of yans

Comment 1 Mathieu Lacage 2009-07-13 03:28:05 EDT

while removing this code, I realized that the realtime simulator implementation actually documents that ::Next is supposed to be used by users of Simulator::RunOne. Although we have no such user for now, I am marking this bug as dependent on bug 631.

Comment 2 Mathieu Lacage 2009-11-23 09:33:23 EST

I am going to mark this method as deprecated and remove it in 3.8

Comment 3 Mathieu Lacage 2012-03-26 08:04:11 EDT

changeset: 369bf20998e8

Comment 4 Tom Henderson 2012-03-28 08:47:36 EDT

MPI depends on it:
[ 824/1729] cxx: src/mpi/model/distributed-simulator-impl.cc -> build/src/mpi/model/distributed-simulator-impl.cc.1.o
../src/mpi/model/distributed-simulator-impl.cc: In member function ‘virtual void ns3::DistributedSimulatorImpl::Run()’:
../src/mpi/model/distributed-simulator-impl.cc:264:29: error: ‘Next’ was not declared in this scope

Comment 5 Mathieu Lacage 2012-03-28 09:03:22 EDT

I simply have no way to build that code so I pushed a tentative fix. changeset: 941d5f5910a5