diff options
author | Josh Stone <jistone@redhat.com> | 2009-08-03 15:49:40 -0700 |
---|---|---|
committer | Josh Stone <jistone@redhat.com> | 2009-08-03 15:49:40 -0700 |
commit | 8b095b454b34e88c04592be6c651153f802eced6 (patch) | |
tree | 9dd9080f259427ddad2ae70de7ba908832b4ef07 /tapset-utrace.cxx | |
parent | 3a4235b9897b75a188753419a29f0616c1686249 (diff) | |
download | systemtap-steved-8b095b454b34e88c04592be6c651153f802eced6.tar.gz systemtap-steved-8b095b454b34e88c04592be6c651153f802eced6.tar.xz systemtap-steved-8b095b454b34e88c04592be6c651153f802eced6.zip |
Add update_visitor::replace
I noticed that most uses of update_visitor::require() were simply
writing the value back to the same place, i.e. foo = require(foo). The
new replace() method just encapsulates that paradigm, so we don't have
the duplication between the LHS and RHS.
* staptree.h (update_visitor::replace): New.
* elaborate.cxx, staptree.cxx, tapset-mark.cxx, tapset-perfmon.cxx,
tapset-procfs.cxx, tapset-utrace.cxx, tapsets.cxx: Update all require
calls that are simply updating the value in-place.
Diffstat (limited to 'tapset-utrace.cxx')
-rw-r--r-- | tapset-utrace.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tapset-utrace.cxx b/tapset-utrace.cxx index 87053f7d..a07e08b8 100644 --- a/tapset-utrace.cxx +++ b/tapset-utrace.cxx @@ -120,7 +120,7 @@ utrace_derived_probe::utrace_derived_probe (systemtap_session &s, { // Expand local variables in the probe body utrace_var_expanding_visitor v (s, l, name, flags); - this->body = v.require (this->body); + v.replace (this->body); target_symbol_seen = v.target_symbol_seen; // If during target-variable-expanding the probe, we added a new block |