[apparmor] [PATCH 02/10] Now that State Cases have been renamed, rename NodeCases back to Cases.
John Johansen
john.johansen at canonical.com
Fri Oct 28 19:19:29 UTC 2011
Signed-off-by: John Johansen <john.johansen at canonical.com>
---
parser/libapparmor_re/expr-tree.h | 24 +++++++++++-------------
parser/libapparmor_re/hfa.cc | 4 ++--
2 files changed, 13 insertions(+), 15 deletions(-)
diff --git a/parser/libapparmor_re/expr-tree.h b/parser/libapparmor_re/expr-tree.h
index 5e9cf1c..930fffc 100644
--- a/parser/libapparmor_re/expr-tree.h
+++ b/parser/libapparmor_re/expr-tree.h
@@ -81,17 +81,15 @@ ostream &operator<<(ostream &os, const NodeSet &state);
* (i.e., following an AnyCharNode or NotCharSetNode). This avoids
* enumerating all the explicit tranitions for default matches.
*/
-typedef struct NodeCases {
+typedef struct Cases {
typedef map<uchar, NodeSet *>::iterator iterator;
iterator begin() { return cases.begin(); }
iterator end() { return cases.end(); }
- NodeCases(): otherwise(0) { }
+ Cases(): otherwise(0) { }
map<uchar, NodeSet *> cases;
NodeSet *otherwise;
-}
-
-NodeCases;
+} Cases;
ostream &operator<<(ostream &os, Node &node);
@@ -205,7 +203,7 @@ public:
ImportantNode(): LeafNode() { }
void compute_firstpos() { firstpos.insert(this); }
void compute_lastpos() { lastpos.insert(this); }
- virtual void follow(NodeCases &cases) = 0;
+ virtual void follow(Cases &cases) = 0;
};
/* common base class for all the different classes that contain
@@ -220,7 +218,7 @@ public:
class CharNode: public CNode {
public:
CharNode(uchar c): c(c) { }
- void follow(NodeCases &cases)
+ void follow(Cases &cases)
{
NodeSet **x = &cases.cases[c];
if (!*x) {
@@ -251,7 +249,7 @@ public:
class CharSetNode: public CNode {
public:
CharSetNode(Chars &chars): chars(chars) { }
- void follow(NodeCases &cases)
+ void follow(Cases &cases)
{
for (Chars::iterator i = chars.begin(); i != chars.end(); i++) {
NodeSet **x = &cases.cases[*i];
@@ -292,7 +290,7 @@ public:
class NotCharSetNode: public CNode {
public:
NotCharSetNode(Chars &chars): chars(chars) { }
- void follow(NodeCases & cases)
+ void follow(Cases &cases)
{
if (!cases.otherwise)
cases.otherwise = new NodeSet;
@@ -305,7 +303,7 @@ public:
* the old otherwise state for the matching characters.
*/
cases.otherwise->insert(followpos.begin(), followpos.end());
- for (NodeCases::iterator i = cases.begin(); i != cases.end();
+ for (Cases::iterator i = cases.begin(); i != cases.end();
i++) {
if (chars.find(i->first) == chars.end())
i->second->insert(followpos.begin(),
@@ -340,12 +338,12 @@ public:
class AnyCharNode: public CNode {
public:
AnyCharNode() { }
- void follow(NodeCases &cases)
+ void follow(Cases &cases)
{
if (!cases.otherwise)
cases.otherwise = new NodeSet;
cases.otherwise->insert(followpos.begin(), followpos.end());
- for (NodeCases::iterator i = cases.begin(); i != cases.end();
+ for (Cases::iterator i = cases.begin(); i != cases.end();
i++)
i->second->insert(followpos.begin(), followpos.end());
}
@@ -372,7 +370,7 @@ public:
*/
}
- void follow(NodeCases &cases __attribute__ ((unused)))
+ void follow(Cases &cases __attribute__ ((unused)))
{
/* Nothing to follow. */
}
diff --git a/parser/libapparmor_re/hfa.cc b/parser/libapparmor_re/hfa.cc
index 9bee40c..79bcac7 100644
--- a/parser/libapparmor_re/hfa.cc
+++ b/parser/libapparmor_re/hfa.cc
@@ -92,7 +92,7 @@ void DFA::update_state_transitions(NodeMap &nodemap, list<State *> &work_queue,
* The resultant transition set is a mapping of characters to
* sets of nodes.
*/
- NodeCases cases;
+ Cases cases;
for (NodeSet::iterator i = state->nodes->begin(); i != state->nodes->end(); i++)
(*i)->follow(cases);
@@ -110,7 +110,7 @@ void DFA::update_state_transitions(NodeMap &nodemap, list<State *> &work_queue,
/* For each transition from *from, check if the set of nodes it
* transitions to already has been mapped to a state
*/
- for (NodeCases::iterator j = cases.begin(); j != cases.end(); j++) {
+ for (Cases::iterator j = cases.begin(); j != cases.end(); j++) {
State *target;
target = find_target_state(nodemap, work_queue, j->second, stats);
--
1.7.5.4
More information about the AppArmor
mailing list