Do my types meet container requirements?

[Dave]

Hello all,

In the code below, can anybody see that I am violating any of the rules that
types must abide by to be held in containers? I get a memory access problem
when a ZoneSetsType destructs...

Thanks,
Dave

struct OnePolygonType
{
unsigned int ModTime;
std::string Pts;
std::string Type;

unsigned int Id; // Unused in comparing objects

// Implement a strict weak ordering
bool operator<(const OnePolygonType &rhs) const
{
if (ModTime < rhs.ModTime)
return true;
else if (ModTime > rhs.ModTime)
return false;

if (Pts < rhs.Pts)
return true;
else if (Pts > rhs.Pts)
return false;

if (Type < rhs.Type)
return true;
else if (Type > rhs.Type)
return false;

return false;
}

bool operator==(cons t OnePolygonType &rhs) const
{
return (
(ModTime == rhs.ModTime) &&
(Pts == rhs.Pts) &&
(Type == rhs.Type)
);
}
};

typedef std::set<OnePol ygonType> PolygonSetType;

struct ZoneSetType
{
PolygonSetType SafeZones;
PolygonSetType HotZones;

// Implement a strict weak ordering
bool operator<(const ZoneSetType &rhs) const
{
if (SafeZones < rhs.SafeZones)
return true;
else if (SafeZones > rhs.SafeZones)
return false;

if (HotZones < rhs.HotZones)
return true;
else if (HotZones > rhs.HotZones)
return false;

return false;
}

bool operator==(cons t ZoneSetType &rhs) const
{
return (
(SafeZones == rhs.SafeZones) &&
(HotZones == rhs.HotZones)
);
}
};

typedef std::map<std::s tring, ZoneSetType> ZoneSetsType;

[Karthik Kumar]

Dave wrote:

Hello all,

In the code below, can anybody see that I am violating any of the rules that
types must abide by to be held in containers? I get a memory access problem
when a ZoneSetsType destructs...

Thanks,
Dave

[code snipped]

In the code you had shown you are not creating any instance of
ZoneSetsType anywhere (statically / dynamically ) .
Also your classes do not seem to have any pointers / files as its
members. (and hence the default copy ctor. / assignment operator /
destructor would be fine). So the problem does not seem to be with the
class definitions but with the client code of the class. Please post the
precise code that you were working on, to understand the problem better.

--
Karthik.

[Dave]

"Karthik Kumar" <ka************ *******@yahoo.c om> wrote in message
news:41c8aaea$1 @darkstar...

Dave wrote:

Hello all,

In the code below, can anybody see that I am violating any of the rules that types must abide by to be held in containers? I get a memory access problem when a ZoneSetsType destructs...

Thanks,
Dave

[code snipped]

In the code you had shown you are not creating any instance of
ZoneSetsType anywhere (statically / dynamically ) .
Also your classes do not seem to have any pointers / files as its
members. (and hence the default copy ctor. / assignment operator /
destructor would be fine). So the problem does not seem to be with the
class definitions but with the client code of the class. Please post the
precise code that you were working on, to understand the problem better.

--
Karthik.

This will be out of context, but here's what the client code is so far (I've
just started developing this function).
As you can see, I'm simply locally declaring and initializing an instance of
the problem type. When ZoneSetsFromMes sage goes out of scope and destructs
upon function exit, I get the memory access problem. Please note that this
happens even if I do not initialize ZoneSetsFromMes sage. Default
constructing it and letting it destruct naturally reproduces the problem.

I tried compiling the code from my original post on Comeau online and it
complained about some concept checks failing. For example:

"concept_checks .h", line 405: error: no operator "!=" matches these operands
operand types are: OnePolygonType != OnePolygonType
if (__a == __b || __a != __b) return __a;
^
detected during:
instantiation of "_Type

_STL_ERROR::__e quality_compara ble_requirement _violation(_
Type, _Type) [with _Type=OnePolygo nType]" at line 555
instantiation of "void
[error snipped]

There were 5 similar errors. I don't understand what C++ rule this is
telling me I'm violating.

Thanks,
Dave

Here's the client code:

void PsrsClient::Han dleUnitZones(co nst unsigned char *&p, size_t &s)
{
UnitZonesIbMess age0 Msg(p, s);
PolygonTransfer Vector_t Polygons = Msg.GetPolygons ();
ZoneSetsType ZoneSetsFromMes sage = CreateZoneSetsT ypeFromMessage( Polygons);

// assert(ZoneSets FromMessage.siz e() == 1);
// ZoneSetType ZoneSet = ZoneSetsFromMes sage.begin()->second;
}

[Andre Kostur]

"Dave" <be***********@ yahoo.com> wrote in
news:10******** *****@news.supe rnews.com:

[snippage]

"concept_checks .h", line 405: error: no operator "!=" matches these
operands
operand types are: OnePolygonType != OnePolygonType
if (__a == __b || __a != __b) return __a;
^
detected during:
instantiation of "_Type
OK... what part of this error message is bad? It's telling you that it
can't find an operator!= that matches the parameters that you're using with
it. Note that in the code you provided earlier I don't see you providing
an operator!=() anywhere.... (I _do_ see an operator==(), but that's a
whole different story....)

_STL_ERROR::__e quality_compara ble_requirement _violation(_
Type, _Type) [with _Type=OnePolygo nType]" at
line 555
instantiation of "void
[error snipped]

Umm... line 555 of what?

[Dave]

"Andre Kostur" <nn******@kostu r.net> wrote in message
news:Xn******** *************** ********@207.35 .177.135...

"Dave" <be***********@ yahoo.com> wrote in
news:10******** *****@news.supe rnews.com:

[snippage]

"concept_checks .h", line 405: error: no operator "!=" matches these
operands
operand types are: OnePolygonType != OnePolygonType
if (__a == __b || __a != __b) return __a;
^
detected during:
instantiation of "_Type
OK... what part of this error message is bad? It's telling you that it
can't find an operator!= that matches the parameters that you're using

with it. Note that in the code you provided earlier I don't see you providing
an operator!=() anywhere.... (I _do_ see an operator==(), but that's a
whole different story....)

Indeed, but nor do I ever use operator!= (my code *does* compile) and nor is
it a requirement of a type storable in an STL container!

_STL_ERROR::__e quality_compara ble_requirement _violation(_
Type, _Type) [with _Type=OnePolygo nType]" at
line 555
instantiation of "void
[error snipped]

Umm... line 555 of what?

[Karthik Kumar]

Dave wrote:

This will be out of context, but here's what the client code is so far (I've
just started developing this function).
As you can see, I'm simply locally declaring and initializing an instance of
the problem type. When ZoneSetsFromMes sage goes out of scope and destructs
upon function exit, I get the memory access problem. Please note that this
happens even if I do not initialize ZoneSetsFromMes sage. Default
constructing it and letting it destruct naturally reproduces the problem.

I tried compiling the code from my original post on Comeau online and it
complained about some concept checks failing. For example:

"concept_checks .h", line 405: error: no operator "!=" matches these operands
operand types are: OnePolygonType != OnePolygonType
if (__a == __b || __a != __b) return __a;
^
detected during:
instantiation of "_Type

_STL_ERROR::__e quality_compara ble_requirement _violation(_
Type, _Type) [with _Type=OnePolygo nType]" at line 555
instantiation of "void
[error snipped]

There were 5 similar errors. I don't understand what C++ rule this is
telling me I'm violating.

Thanks,
Dave

Here's the client code:

void PsrsClient::Han dleUnitZones(co nst unsigned char *&p, size_t &s)
{
UnitZonesIbMess age0 Msg(p, s);
PolygonTransfer Vector_t Polygons = Msg.GetPolygons ();
ZoneSetsType ZoneSetsFromMes sage = CreateZoneSetsT ypeFromMessage( Polygons);

// assert(ZoneSets FromMessage.siz e() == 1);
// ZoneSetType ZoneSet = ZoneSetsFromMes sage.begin()->second;
}

There apppears to be two more objects in the local function.
Are you sure it is the ZoneSetsType that is causing the problem
and not the other objects when go out of scope, (say, Msg of type
UnitZonesIbMess age0 ).

--
Karthik.

[Dave]

"Karthik Kumar" <ka************ *******@yahoo.c om> wrote in message
news:41c8bf81$1 @darkstar...

Dave wrote:

This will be out of context, but here's what the client code is so far (I've just started developing this function).
As you can see, I'm simply locally declaring and initializing an instance of the problem type. When ZoneSetsFromMes sage goes out of scope and destructs upon function exit, I get the memory access problem. Please note that this happens even if I do not initialize ZoneSetsFromMes sage. Default
constructing it and letting it destruct naturally reproduces the problem.
I tried compiling the code from my original post on Comeau online and it
complained about some concept checks failing. For example:

"concept_checks .h", line 405: error: no operator "!=" matches these operands operand types are: OnePolygonType != OnePolygonType
if (__a == __b || __a != __b) return __a;
^
detected during:
instantiation of "_Type

_STL_ERROR::__e quality_compara ble_requirement _violation(_
Type, _Type) [with _Type=OnePolygo nType]" at line 555 instantiation of "void
[error snipped]

There were 5 similar errors. I don't understand what C++ rule this is
telling me I'm violating.

Thanks,
Dave

Here's the client code:

void PsrsClient::Han dleUnitZones(co nst unsigned char *&p, size_t &s)
{
UnitZonesIbMess age0 Msg(p, s);
PolygonTransfer Vector_t Polygons = Msg.GetPolygons ();
ZoneSetsType ZoneSetsFromMes sage = CreateZoneSetsT ypeFromMessage( Polygons);
// assert(ZoneSets FromMessage.siz e() == 1);
// ZoneSetType ZoneSet = ZoneSetsFromMes sage.begin()->second;
}

There apppears to be two more objects in the local function.
Are you sure it is the ZoneSetsType that is causing the problem
and not the other objects when go out of scope, (say, Msg of type
UnitZonesIbMess age0 ).

--
Karthik.

Actually, I believe I've now found out where the problem lies.

It's a VC++ 6.0 thing. The call to CreateZoneSetsT ypeFromMessage( ) is
across a DLL boundary, and each DLL has its own heap in this environment.
So, memory is allocated in one heap and deallocated in another. Ouch!

I guess I'll have to terminate this thread now since we've gotten into a
particular implementation' s quirks...

Thanks for the help!

Dave

[Andre Kostur]

"Dave" <be***********@ yahoo.com> wrote in
news:10******** *****@news.supe rnews.com:

"Andre Kostur" <nn******@kostu r.net> wrote in message
news:Xn******** *************** ********@207.35 .177.135...

"Dave" <be***********@ yahoo.com> wrote in
news:10******** *****@news.supe rnews.com:

[snippage]

> "concept_checks .h", line 405: error: no operator "!=" matches these
> operands
> operand types are: OnePolygonType != OnePolygonType
> if (__a == __b || __a != __b) return __a;
> ^
> detected during:
> instantiation of "_Type

OK... what part of this error message is bad? It's telling you that
it can't find an operator!= that matches the parameters that you're
using

with

it. Note that in the code you provided earlier I don't see you
providing an operator!=() anywhere.... (I _do_ see an operator==(),
but that's a whole different story....)

Indeed, but nor do I ever use operator!= (my code *does* compile) and
nor is it a requirement of a type storable in an STL container!

Huh? What's this error message about line 405 of "concept_checks .h" ?

[Pete Becker]

Dave wrote:

It's a VC++ 6.0 thing. The call to CreateZoneSetsT ypeFromMessage( ) is
across a DLL boundary, and each DLL has its own heap in this environment.
So, memory is allocated in one heap and deallocated in another.

Only if you build it that way.

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)