Bug 629 – Wimax doesn't compile with gcc-4.3.x and gcc-4.4.x versions

Bug 629 - Wimax doesn't compile with gcc-4.3.x and gcc-4.4.x versions


Summary:	Wimax doesn't compile with gcc-4.3.x and gcc-4.4.x versions

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	wimax
Version:	pre-release
Hardware:	All Linux

Importance:	P5 normal
Assigned To:	ns-bugs

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-07-08 11:24 UTC by Faker Moatamri
Modified:	2009-08-07 03:29 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
Correct the template call to sendBurst with Simulator::Schedule (2.29 KB, patch) 2009-07-08 11:43 UTC, Faker Moatamri	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 Faker Moatamri 2009-07-08 11:24:53 UTC

two errors comes out when we compile with those versions of gcc:
First one is easy to fix:
from debug/ns3/wimax-ss-net-device.h:30,
from debug/ns3/wimax-bs-net-device.h:30,
from ../src/helper/wimax-helper.h:30,
from ../src/helper/wimax-helper.cc:23:
debug/ns3/IPCS_Bind_parameter_manager.h:40: error: multiple parameters named ‘src_ip_adr’
Waf: Leaving directory `/auto/elvis/u/elvis/1/user/fmoatamr/src/wimax/build' 

and the second one is harder to fix:
debug/ns3/make-event.h: In member function ‘void ns3::MakeEvent(MEM, OBJ, T1, T2, T3)::EventMemberImpl3::Notify() [with MEM = void (ns3::WimaxSubscriberStationNetDevice::*)(uint8_t, uint16_t, ns3::Ptr<ns3::WimaxConnection>, ns3::MacHeaderType::HeaderType), OBJ = ns3::WimaxSubscriberStationNetDevice*, T1 = unsigned char, T2 = short unsigned int, T3 = ns3::Ptr<ns3::WimaxConnection>]’:
../src/devices/wimax/wimax-ss-net-device.cc:1059:   instantiated from here
debug/ns3/make-event.h:167: error: too few arguments to function

Comment 1 Faker Moatamri 2009-07-08 11:42:19 UTC

For the first problem it was easy to fix, just replace src_ip_adr by dst_ip_adr
For the second error it seemed a bit complicated since the error doesn't say where the problem is, as usual with C++.
Finally I found that there is a Simulator::Schedule call to SendBurst function that was missing one operand, Amine, can you please check the patch (it's short) and push it to your repository?

Comment 2 Faker Moatamri 2009-07-08 11:43:42 UTC

Created attachment 519 [details]
Correct the template call to sendBurst with Simulator::Schedule