std::list pass by reference initialization error

[Ray D.]

Hey all,

I'm trying to pass a list into a function to edit it but when I
compile using g++ I continue to get the following error:

maintainNeighbo rs.cpp:104: error: invalid initialization of non-const
reference of type 'std::list<Host ID, std::allocator< HostID&' from a
temporary of type 'std::list<Host ID, std::allocator< HostID*'

helpers.cpp:99: error: in passing argument 1 of `void
CheckIfNeighbor sHaveSentHello( std::list<HostI D, std::allocator< HostID>

>&)'

The function is shown below:

void CheckIfNeighbor sHaveSentHello( std::list<struc t HostID>
&Neighbors)
{
std::list<struc t HostID>::iterat or it;
std::list<struc t HostID>::iterat or LastIt;
struct timeb TimeBuffer;
ftime( &TimeBuffer );

it=Neighbors.be gin();
while (it!=Neighbors. end())
{
int Del=0;
if (TimeBuffer.tim e - it->LastHelloRec 40)
Del=1;
LastIt = it;
++it;
if (Del==1)
Neighbors.erase (LastIt);
}
}

And the objects contained in the list are shown below, along with how
it is defined and the function call itself:

struct HostID {
char IP[16];
int Port;
int LastHelloRec;
int LastHelloSent;
};

std::list<struc t HostIDActiveNei ghbors;
CheckIfNeighbor sHaveSentHello( &ActiveNeighbor s);
My guess is that it has something to do with the iterator, but I've
been stuck on this for a while now and I figure a more experienced
person could guide me in the right direction. Thanks in advance for
any help!

[kwikius]

On Apr 30, 8:58*pm, "Ray D." <ray.delvecc... @gmail.comwrote :

Hey all,

I'm trying to pass a list into a function to edit it but when I
compile using g++ I continue to get the following error:

maintainNeighbo rs.cpp:104: error: invalid initialization of non-const
reference of type 'std::list<Host ID, std::allocator< HostID&' from a
temporary of type 'std::list<Host ID, std::allocator< HostID*'

<...>

std::list<struc t HostIDActiveNei ghbors;
CheckIfNeighbor sHaveSentHello( &ActiveNeighbor s);

^^^
try :

CheckIfNeighbor sHaveSentHello( ActiveNeighbors );

(passing as a reference rather than as a pointer )

(but not tested)

regards
Andy Little

[utab]

On Apr 30, 9:58 pm, "Ray D." <ray.delvecc... @gmail.comwrote :

Hey all,

I'm trying to pass a list into a function to edit it but when I
compile using g++ I continue to get the following error:

maintainNeighbo rs.cpp:104: error: invalid initialization of non-const
reference of type 'std::list<Host ID, std::allocator< HostID&' from a
temporary of type 'std::list<Host ID, std::allocator< HostID*'

helpers.cpp:99: error: in passing argument 1 of `void
CheckIfNeighbor sHaveSentHello( std::list<HostI D, std::allocator< HostID>

&)'

The function is shown below:

void CheckIfNeighbor sHaveSentHello( std::list<struc t HostID>
&Neighbors)
{
std::list<struc t HostID>::iterat or it;
std::list<struc t HostID>::iterat or LastIt;
struct timeb TimeBuffer;
ftime( &TimeBuffer );

it=Neighbors.be gin();
while (it!=Neighbors. end())
{
int Del=0;
if (TimeBuffer.tim e - it->LastHelloRec 40)
Del=1;
LastIt = it;
++it;
if (Del==1)
Neighbors.erase (LastIt);
}

}

And the objects contained in the list are shown below, along with how
it is defined and the function call itself:

struct HostID {
char IP[16];
int Port;
int LastHelloRec;
int LastHelloSent;

};

std::list<struc t HostIDActiveNei ghbors;
CheckIfNeighbor sHaveSentHello( &ActiveNeighbor s);

Here is the problem, expecting a reference, not a pointer ...
Reference
in the parameter list becomes an alias for ActiveNeighbors , but you
are
passing the address of the list, which is the cause I guess...

My guess is that it has something to do with the iterator, but I've
been stuck on this for a while now and I figure a more experienced
person could guide me in the right direction. Thanks in advance for
any help!

[Jim Langston]

utab wrote:

On Apr 30, 9:58 pm, "Ray D." <ray.delvecc... @gmail.comwrote :

>Hey all,

I'm trying to pass a list into a function to edit it but when I
compile using g++ I continue to get the following error:

maintainNeighb ors.cpp:104: error: invalid initialization of non-const
reference of type 'std::list<Host ID, std::allocator< HostID&' from
a temporary of type 'std::list<Host ID, std::allocator< HostID*'

helpers.cpp:99 : error: in passing argument 1 of `void
CheckIfNeighbo rsHaveSentHello (std::list<Host ID,
std::allocator <HostID>

>>&)'

The function is shown below:

void CheckIfNeighbor sHaveSentHello( std::list<struc t HostID>
&Neighbors)
{
std::list<struc t HostID>::iterat or it;
std::list<struc t HostID>::iterat or LastIt;
struct timeb TimeBuffer;
ftime( &TimeBuffer );

it=Neighbors.be gin();
while (it!=Neighbors. end())
{
int Del=0;
if (TimeBuffer.tim e - it->LastHelloRec 40)
Del=1;
LastIt = it;
++it;
if (Del==1)
Neighbors.erase (LastIt);
}

}

And the objects contained in the list are shown below, along with how
it is defined and the function call itself:

struct HostID {
char IP[16];
int Port;
int LastHelloRec;
int LastHelloSent;

};

std::list<stru ct HostIDActiveNei ghbors;
CheckIfNeighbo rsHaveSentHello (&ActiveNeighbo rs);

Here is the problem, expecting a reference, not a pointer ...
Reference
in the parameter list becomes an alias for ActiveNeighbors , but you
are
passing the address of the list, which is the cause I guess...

In other words, try changing it to:
CheckIfNeighbor sHaveSentHello( ActiveNeighbors );

References work on the instanct name, not the address of the instance.

>My guess is that it has something to do with the iterator, but I've
been stuck on this for a while now and I figure a more experienced
person could guide me in the right direction. Thanks in advance for
any help!

--
Jim Langston
ta*******@rocke tmail.com