[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