[Xorp-cvs] XORP cvs commit: xorp/policy xorp/policy/backend xorp/policy/common xorp/policy/test
Pavlin Radoslavov
pavlin at icir.org
Wed May 24 22:50:02 PDT 2006
CVSROOT: /usr/local/share/doc/apache/cvs
Module name: xorp
Changes by: pavlin at xorpc.icir.org 2006-05-25 05:50:02 UTC
XORP CVS repository
Modified files:
policy semantic_varrw.cc semantic_varrw.hh
policy/backend iv_exec.cc single_varrw.cc single_varrw.hh
policy/common varrw.hh
policy/test file_varrw.cc file_varrw.hh
Log message:
Reverse-back the previous commit.
The original code's semantics was fine, because the writing/reading
of variables was happening by utilizing a local policy cache.
The reading was actually returning the new value as written to the
cache by an earlier policy term.
The sync() was necessary at the end of a policy statement to write
the final values to the underlying structure.
In fact, the previous commit broke the (desired) syntax, because
each sync() will throw-away the cache, so the read (from the
empty cache) wouldn't return the expected value.
Requested by: Andrea Bittau
Revision Changes Path
1.10 +3 -16; commitid: a45a447544397ea6; xorp/policy/backend/iv_exec.cc
1.13 +1 -5; commitid: a45a447544397ea6; xorp/policy/backend/single_varrw.cc
1.9 +1 -3; commitid: a45a447544397ea6; xorp/policy/backend/single_varrw.hh
1.10 +1 -2; commitid: a45a447544397ea6; xorp/policy/common/varrw.hh
1.8 +1 -3; commitid: a45a447544397ea6; xorp/policy/semantic_varrw.cc
1.6 +1 -3; commitid: a45a447544397ea6; xorp/policy/semantic_varrw.hh
1.8 +2 -8; commitid: a45a447544397ea6; xorp/policy/test/file_varrw.cc
1.6 +1 -1; commitid: a45a447544397ea6; xorp/policy/test/file_varrw.hh
More information about the Xorp-cvs
mailing list