1
2- rewrite all the translators more cleanly. Whereas we used to do :
3
4
5 switch(operator)
6{
7   case ACCEPT_ALL :
8     ...
9    break;
10    case DENY_ALL :
11     ...
12    break;
13    case ACCEPT_INCOMING :
14     ...
15    break;
16}
17
18we should use a more modular approach :
19
20
21 if(operator & ACCEPT)printf("accept ");
22 if(operator & DENY)printf("deny ");
23 if(operator & LOG) printf("log ");
24 if(operator & ONE_WAY)printf("from %s ", ip);
25 .
26 .
27 .
28
29
30
31
32
33