summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOndrej Filip <feela@network.cz>2010-08-03 15:14:26 +0200
committerOndrej Filip <feela@network.cz>2010-08-03 15:16:25 +0200
commit4733b49ed6ec11669061e15680417961d13c7a61 (patch)
tree2da1227f13c4004814f0ac231541eec0d3734b61
parent2c9033afd5ce5e99255d248fb065e94534405da7 (diff)
downloadbird-4733b49ed6ec11669061e15680417961d13c7a61.tar
bird-4733b49ed6ec11669061e15680417961d13c7a61.zip
Syntax of sets improved.
-rw-r--r--doc/bird.sgml3
-rw-r--r--filter/config.Y30
-rw-r--r--filter/test.conf21
3 files changed, 39 insertions, 15 deletions
diff --git a/doc/bird.sgml b/doc/bird.sgml
index 266c449..c46ea4f 100644
--- a/doc/bird.sgml
+++ b/doc/bird.sgml
@@ -707,7 +707,8 @@ incompatible with each other (that is to prevent you from shooting in the foot).
but you can't modify them. Literals of type <cf>int set</cf> look like <cf>
[ 1, 2, 5..7 ]</cf>. As you can see, both simple values and ranges are permitted in
sets. For pair sets, expressions like <cf/(123,*)/ can be used to denote ranges (in
- that case <cf/(123,0)..(123,65535)/).
+ that case <cf/(123,0)..(123,65535)/). You can also use <cf/(123,5..100/ for range
+ <cf/(123,5)..(123,100)/.
Sets of prefixes are special: their literals does not allow ranges, but allows
prefix patterns that are written as <cf><M>ipaddress</M>/<M>pxlen</M>{<M>low</M>,<M>high</M>}</cf>.
diff --git a/filter/config.Y b/filter/config.Y
index 0b507ff..698f920 100644
--- a/filter/config.Y
+++ b/filter/config.Y
@@ -45,7 +45,7 @@ CF_KEYWORDS(FUNCTION, PRINT, PRINTN, UNSET, RETURN,
%type <x> term block cmds cmds_int cmd function_body constant print_one print_list var_list var_listn dynamic_attr static_attr function_call symbol dpair bgp_path_expr
%type <f> filter filter_body where_filter
-%type <i> type break_command cpair
+%type <i> type break_command
%type <e> set_item set_items switch_body
%type <trie> fprefix_set
%type <v> set_atom fprefix fprefix_s fipa
@@ -221,13 +221,6 @@ block:
;
/*
- * Simple types, their bison value is int
- */
-cpair:
- '(' NUM ',' NUM ')' { $$ = make_pair($2, $4); }
- ;
-
-/*
* Complex types, their bison value is struct f_val
*/
fipa:
@@ -237,13 +230,30 @@ fipa:
set_atom:
NUM { $$.type = T_INT; $$.val.i = $1; }
| RTRID { $$.type = T_QUAD; $$.val.i = $1; }
- | cpair { $$.type = T_PAIR; $$.val.i = $1; }
| fipa { $$ = $1; }
| ENUM { $$.type = $1 >> 16; $$.val.i = $1 & 0xffff; }
;
set_item:
- '(' NUM ',' '*' ')' {
+ '(' expr ',' expr ')' {
+ $$ = f_new_tree();
+ $$->from.type = $$->to.type = T_PAIR;
+ $$->from.val.i = make_pair($2, $4);
+ $$->to.val.i = make_pair($2, $4);
+ }
+ | '(' expr ',' expr '.' '.' expr ')' {
+ $$ = f_new_tree();
+ $$->from.type = $$->to.type = T_PAIR;
+ $$->from.val.i = make_pair($2, $4);
+ $$->to.val.i = make_pair($2, $7);
+ }
+ | '(' expr ',' expr ')' '.' '.' '(' expr ',' expr ')' {
+ $$ = f_new_tree();
+ $$->from.type = $$->to.type = T_PAIR;
+ $$->from.val.i = make_pair($2, $4);
+ $$->to.val.i = make_pair($9, $11);
+ }
+ | '(' expr ',' '*' ')' {
$$ = f_new_tree();
$$->from.type = $$->to.type = T_PAIR;
$$->from.val.i = make_pair($2, 0);
diff --git a/filter/test.conf b/filter/test.conf
index 4814396..18121c4 100644
--- a/filter/test.conf
+++ b/filter/test.conf
@@ -9,6 +9,7 @@ router id 62.168.0.1;
define xyzzy = (120+10);
define '1a-a1' = (20+10);
+define one = 1;
function 'mkpair-a'(int a)
@@ -76,14 +77,26 @@ clist l;
print "Should be true: ", p2 ~ pm1, " ", p2, " ", pm1;
l = - empty -;
- l = add( l, (1,2) );
- l = add( l, (2,3) );
+ l = add( l, (one,2) );
+ l = add( l, (2,one+2) );
print "Community list (1,2) (2,3) ", l;
- print "Should be true: ", (2,3) ~ l, " ", l ~ [(1,*)], " ", l ~ [(2,3)];
+ print "Should be true: ", (2,3) ~ l, " ", l ~ [(1,*)], " ", l ~ [(2,3)]," ", l ~ [(2,2..3)], " ", l ~ [(1,1..2)], " ", l ~ [(1,1)..(1,2)];
l = add( l, (2,5) );
+ l = add( l, (5,one) );
+ l = add( l, (6,one) );
+ l = add( l, (one,one) );
+ l = delete( l, [(5,1),(6,one),(one,1)] );
+ l = delete( l, [(5,one),(6,one)] );
l = delete( l, [(2,*)] );
print "Community list (1,2) ", l;
- print "Should be false: ", (2,3) ~ l, " ", l ~ [(2,*)];
+ print "Should be false: ", (2,3) ~ l, " ", l ~ [(2,*)], " ", l ~ [(one,3..6)];
+ l = add( l, (3,one) );
+ l = add( l, (one+one+one,one+one) );
+ l = add( l, (3,3) );
+ l = add( l, (3,4) );
+ l = add( l, (3,5) );
+ l = delete( l, [(3,2..4)] );
+ print "Community list (1,2) (3,1) (3,5) ", l;
}
function bla()