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