mirrors/apparmor
1
0
Fork 0
mirror of https://gitlab.com/apparmor/apparmor.git synced 2025-03-04 08:24:42 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Older C++ compilers complain about the use of a class with a non trivial

Browse source 
constructor in a union.  Change the ProtoState class to use an init fn
instead of a constructor.
This commit is contained in:
John Johansen 2012-05-30 14:31:41 -07:00
parent 2347b6628d
commit 41b454f2e5
2 changed files with 14 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
parser/libapparmor_re/hfa.cc
View file

@ -101,7 +101,8 @@ State *DFA::add_new_state(NodeSet *nodes, State *other)
nnodev = nnodes_cache.insert(nnodes);
anodes = anodes_cache.insert(anodes);
ProtoState proto(nnodev, anodes);
ProtoState proto;
proto.init(nnodev, anodes);
State *state = new State(node_map.size(), proto, other);
pair<NodeMap::iterator,bool> x = node_map.insert(proto, state);
if (x.second == false) {

16
parser/libapparmor_re/hfa.h
View file

@ -310,7 +310,15 @@ public:
hashedNodeVec *nnodes;
NodeSet *anodes;
ProtoState(hashedNodeVec *n, NodeSet *a = NULL): nnodes(n), anodes(a) { };
/* init is used instead of a constructor because ProtoState is used
* in a union
*/
void init(hashedNodeVec *n, NodeSet *a = NULL)
{
nnodes = n;
anodes = a;
}
bool operator<(ProtoState const &rhs)const
{
if (nnodes == rhs.nnodes)
@ -338,7 +346,7 @@ public:
* parition: Is a temporary work variable used during dfa minimization.
* it can be replaced with a map, but that is slower and uses more
* memory.
* nodes: Is a temporary work variable used during dfa creation. It can
* proto: Is a temporary work variable used during dfa creation. It can
* be replaced by using the nodemap, but that is slower
*/
class State {
@ -379,8 +387,8 @@ public:
/* temp storage for State construction */
union {
Partition *partition;
ProtoState proto;
Partition *partition; /* used during minimization */
ProtoState proto; /* used during creation */
};
};