aboutsummaryrefslogtreecommitdiffstats
path: root/host/include
diff options
context:
space:
mode:
authorMartin Braun <martin.braun@ettus.com>2017-02-13 13:31:52 -0800
committerMartin Braun <martin.braun@ettus.com>2017-02-13 13:33:04 -0800
commit180d5f3ad9db94bc61688014c0a52eeb5b649ee0 (patch)
treea7910fe66db6aec1ec79d7c098c7058f2d133a8c /host/include
parentb107b4604cedd1ea91b45392c979d179a15d5738 (diff)
downloaduhd-180d5f3ad9db94bc61688014c0a52eeb5b649ee0.tar.gz
uhd-180d5f3ad9db94bc61688014c0a52eeb5b649ee0.tar.bz2
uhd-180d5f3ad9db94bc61688014c0a52eeb5b649ee0.zip
uhd: Undo some of the BOOST_FOREACH() replacements
In public headers, we're going back to BOOST_FOREACH. Thus, not every user needs to build their own project with the same compilers that we do.
Diffstat (limited to 'host/include')
-rw-r--r--host/include/uhd/property_tree.ipp5
-rw-r--r--host/include/uhd/rfnoc/node_ctrl_base.ipp4
-rw-r--r--host/include/uhd/types/dict.ipp17
-rw-r--r--host/include/uhd/utils/assert_has.ipp3
-rw-r--r--host/include/uhd/utils/soft_register.hpp23
5 files changed, 28 insertions, 24 deletions
diff --git a/host/include/uhd/property_tree.ipp b/host/include/uhd/property_tree.ipp
index ef63ced24..6ed1056e6 100644
--- a/host/include/uhd/property_tree.ipp
+++ b/host/include/uhd/property_tree.ipp
@@ -19,6 +19,7 @@
#define INCLUDED_UHD_PROPERTY_TREE_IPP
#include <uhd/exception.hpp>
+#include <boost/foreach.hpp>
#include <boost/scoped_ptr.hpp>
#include <vector>
@@ -71,14 +72,14 @@ public:
void _set_coerced(const T &value){
init_or_set_value(_coerced_value, value);
- for(typename property<T>::subscriber_type &csub: _coerced_subscribers){
+ BOOST_FOREACH(typename property<T>::subscriber_type &csub, _coerced_subscribers){
csub(get_value_ref(_coerced_value)); //let errors propagate
}
}
property<T> &set(const T &value){
init_or_set_value(_value, value);
- for(typename property<T>::subscriber_type &dsub: _desired_subscribers){
+ BOOST_FOREACH(typename property<T>::subscriber_type &dsub, _desired_subscribers){
dsub(get_value_ref(_value)); //let errors propagate
}
if (not _coercer.empty()) {
diff --git a/host/include/uhd/rfnoc/node_ctrl_base.ipp b/host/include/uhd/rfnoc/node_ctrl_base.ipp
index 2492031be..d300f72a7 100644
--- a/host/include/uhd/rfnoc/node_ctrl_base.ipp
+++ b/host/include/uhd/rfnoc/node_ctrl_base.ipp
@@ -45,7 +45,7 @@ namespace uhd {
while (iters++ < MAX_ITER) {
next_q.clear();
- for(const sptr &this_node: search_q) {
+ BOOST_FOREACH(const sptr &this_node, search_q) {
// Add this node to the list of explored nodes
explored.insert(this_node);
// Create set of all child nodes of this_node that are not in explored:
@@ -98,7 +98,7 @@ namespace uhd {
std::vector< boost::shared_ptr<T> > descendant_rate_nodes = _find_child_node<T, downstream>();
value_type ret_val = NULL_VALUE;
std::string first_node_id;
- for(const boost::shared_ptr<T> &node: descendant_rate_nodes) {
+ BOOST_FOREACH(const boost::shared_ptr<T> &node, descendant_rate_nodes) {
if (exclude_nodes.count(node)) {
continue;
}
diff --git a/host/include/uhd/types/dict.ipp b/host/include/uhd/types/dict.ipp
index b9d367a3c..5fd4b536e 100644
--- a/host/include/uhd/types/dict.ipp
+++ b/host/include/uhd/types/dict.ipp
@@ -19,6 +19,7 @@
#define INCLUDED_UHD_TYPES_DICT_IPP
#include <uhd/exception.hpp>
+#include <boost/foreach.hpp>
#include <boost/format.hpp>
#include <boost/lexical_cast.hpp>
#include <typeinfo>
@@ -60,7 +61,7 @@ namespace uhd{
template <typename Key, typename Val>
std::vector<Key> dict<Key, Val>::keys(void) const{
std::vector<Key> keys;
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
keys.push_back(p.first);
}
return keys;
@@ -69,7 +70,7 @@ namespace uhd{
template <typename Key, typename Val>
std::vector<Val> dict<Key, Val>::vals(void) const{
std::vector<Val> vals;
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
vals.push_back(p.second);
}
return vals;
@@ -77,7 +78,7 @@ namespace uhd{
template <typename Key, typename Val>
bool dict<Key, Val>::has_key(const Key &key) const{
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
if (p.first == key) return true;
}
return false;
@@ -85,7 +86,7 @@ namespace uhd{
template <typename Key, typename Val>
const Val &dict<Key, Val>::get(const Key &key, const Val &other) const{
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
if (p.first == key) return p.second;
}
return other;
@@ -93,7 +94,7 @@ namespace uhd{
template <typename Key, typename Val>
const Val &dict<Key, Val>::get(const Key &key) const{
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
if (p.first == key) return p.second;
}
throw key_not_found<Key, Val>(key);
@@ -106,7 +107,7 @@ namespace uhd{
template <typename Key, typename Val>
const Val &dict<Key, Val>::operator[](const Key &key) const{
- for(const pair_t &p: _map){
+ BOOST_FOREACH(const pair_t &p, _map){
if (p.first == key) return p.second;
}
throw key_not_found<Key, Val>(key);
@@ -114,7 +115,7 @@ namespace uhd{
template <typename Key, typename Val>
Val &dict<Key, Val>::operator[](const Key &key){
- for(pair_t &p: _map){
+ BOOST_FOREACH(pair_t &p, _map){
if (p.first == key) return p.second;
}
_map.push_back(std::make_pair(key, Val()));
@@ -137,7 +138,7 @@ namespace uhd{
template <typename Key, typename Val>
void dict<Key, Val>::update(const dict<Key, Val> &new_dict, bool fail_on_conflict)
{
- for(const Key &key: new_dict.keys()) {
+ BOOST_FOREACH(const Key &key, new_dict.keys()) {
if (fail_on_conflict and has_key(key) and get(key) != new_dict[key]) {
throw uhd::value_error(str(
boost::format("Option merge conflict: %s:%s != %s:%s")
diff --git a/host/include/uhd/utils/assert_has.ipp b/host/include/uhd/utils/assert_has.ipp
index 6457cf2ee..974eea5a5 100644
--- a/host/include/uhd/utils/assert_has.ipp
+++ b/host/include/uhd/utils/assert_has.ipp
@@ -22,6 +22,7 @@
#include <uhd/exception.hpp>
#include <boost/format.hpp>
#include <boost/lexical_cast.hpp>
+#include <boost/foreach.hpp>
namespace uhd{
@@ -33,7 +34,7 @@ namespace uhd{
if (uhd::has(range, value)) return;
std::string possible_values = "";
size_t i = 0;
- for(const T &v: range){
+ BOOST_FOREACH(const T &v, range){
if (i++ > 0) possible_values += ", ";
possible_values += boost::lexical_cast<std::string>(v);
}
diff --git a/host/include/uhd/utils/soft_register.hpp b/host/include/uhd/utils/soft_register.hpp
index d3a1e0943..2870ad595 100644
--- a/host/include/uhd/utils/soft_register.hpp
+++ b/host/include/uhd/utils/soft_register.hpp
@@ -27,6 +27,7 @@
#include <boost/thread/locks.hpp>
#include <boost/unordered_map.hpp>
#include <boost/tokenizer.hpp>
+#include <boost/foreach.hpp>
#include <boost/lexical_cast.hpp>
#include <list>
@@ -478,7 +479,7 @@ public:
*/
void initialize(wb_iface& iface, bool sync = false) {
boost::lock_guard<boost::mutex> lock(_mutex);
- for(soft_register_base* reg: _reglist) {
+ BOOST_FOREACH(soft_register_base* reg, _reglist) {
reg->initialize(iface, sync);
}
}
@@ -490,7 +491,7 @@ public:
*/
void flush() {
boost::lock_guard<boost::mutex> lock(_mutex);
- for(soft_register_base* reg: _reglist) {
+ BOOST_FOREACH(soft_register_base* reg, _reglist) {
reg->flush();
}
}
@@ -502,7 +503,7 @@ public:
*/
void refresh() {
boost::lock_guard<boost::mutex> lock(_mutex);
- for(soft_register_base* reg: _reglist) {
+ BOOST_FOREACH(soft_register_base* reg, _reglist) {
reg->refresh();
}
}
@@ -526,7 +527,7 @@ public:
*/
virtual std::vector<std::string> enumerate() const {
std::vector<std::string> temp;
- for(const regmap_t::value_type& reg: _regmap) {
+ BOOST_FOREACH(const regmap_t::value_type& reg, _regmap) {
temp.push_back(_name + "/" + reg.first);
}
return temp;
@@ -622,8 +623,8 @@ public:
{
//Turn the slash separated path string into tokens
std::list<std::string> tokens;
- for(
- const std::string& node:
+ BOOST_FOREACH(
+ const std::string& node,
boost::tokenizer< boost::char_separator<char> >(path, boost::char_separator<char>("/")))
{
tokens.push_back(node);
@@ -632,7 +633,7 @@ public:
(tokens.size() > 1 && _name == "")) { //If this is a top-level DB
if (_name != "") tokens.pop_front();
if (tokens.size() == 2) { //2 tokens => regmap/register path
- for(const soft_regmap_accessor_t* regmap: _regmaps) {
+ BOOST_FOREACH(const soft_regmap_accessor_t* regmap, _regmaps) {
if (regmap->get_name() == tokens.front()) {
return regmap->lookup(tokens.back());
}
@@ -641,11 +642,11 @@ public:
} else if (not _regmap_dbs.empty()) { //>2 tokens => <1 or more dbs>/regmap/register
//Reconstruct path from tokens
std::string newpath;
- for(const std::string& node: tokens) {
+ BOOST_FOREACH(const std::string& node, tokens) {
newpath += ("/" + node);
}
//Dispatch path to hierarchical DB
- for(const soft_regmap_accessor_t* db: _regmap_dbs) {
+ BOOST_FOREACH(const soft_regmap_accessor_t* db, _regmap_dbs) {
try {
return db->lookup(newpath.substr(1));
} catch (std::exception&) {
@@ -662,11 +663,11 @@ public:
*/
virtual std::vector<std::string> enumerate() const {
std::vector<std::string> paths;
- for(const soft_regmap_accessor_t* regmap: _regmaps) {
+ BOOST_FOREACH(const soft_regmap_accessor_t* regmap, _regmaps) {
const std::vector<std::string>& regs = regmap->enumerate();
paths.insert(paths.end(), regs.begin(), regs.end());
}
- for(const soft_regmap_accessor_t* db: _regmap_dbs) {
+ BOOST_FOREACH(const soft_regmap_accessor_t* db, _regmap_dbs) {
const std::vector<std::string>& regs = db->enumerate();
paths.insert(paths.end(), regs.begin(), regs.end());
}