From 2e60fe3cfd5cf5305888fcca0ae19700d808bb23 Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Fri, 31 Jan 2020 10:49:53 +0100 Subject: [PATCH] Cosmetics: fix "Malformed whitespace in C++" spotted by codefactor.io. --- contrib/benchmarking_code_block/bench.h | 2 -- docs/source/tuto_s4u/master-workers-lab1.cpp | 1 - docs/source/tuto_s4u/master-workers-lab2.cpp | 1 - examples/deprecated/msg/dht-kademlia/node.h | 1 - examples/s4u/actor-lifetime/s4u-actor-lifetime.cpp | 1 - examples/s4u/app-bittorrent/s4u-peer.hpp | 1 + .../app-masterworkers/s4u-app-masterworkers-class.cpp | 1 - examples/s4u/async-waituntil/s4u-async-waituntil.cpp | 2 +- .../smpi_s4u_masterslave/masterslave_mailbox_smpi.cpp | 1 - include/simgrid/kernel/routing/NetPoint.hpp | 1 - include/simgrid/s4u/Activity.hpp | 1 + include/simgrid/s4u/Storage.hpp | 1 + include/simgrid/simix/blocking_simcall.hpp | 1 + include/xbt/future.hpp | 1 + src/bindings/lua/lua_utils.cpp | 3 --- src/bindings/lua/simgrid_lua.cpp | 4 ++-- src/instr/instr_paje_containers.hpp | 1 + src/kernel/context/ContextBoost.cpp | 1 - src/kernel/resource/profile/Profile_test.cpp | 2 -- src/kernel/routing/ClusterZone.cpp | 2 +- src/mc/AddressSpace.hpp | 2 +- src/mc/ModelChecker.hpp | 2 +- src/mc/checker/Checker.hpp | 1 + src/mc/checker/CommunicationDeterminismChecker.hpp | 1 + src/mc/mc_comm_pattern.cpp | 1 - src/mc/mc_global.cpp | 1 - src/mc/mc_record.cpp | 2 +- src/mc/mc_smx.cpp | 2 +- src/mc/sosp/ChunkedData.cpp | 1 - src/mc/sosp/Snapshot_test.cpp | 1 - src/msg/msg_global.cpp | 10 ++++++---- src/surf/network_ib.hpp | 1 - src/surf/ns3/ns3_simulator.cpp | 2 +- src/xbt/config_test.cpp | 1 - src/xbt/mmalloc/mmprivate.h | 2 -- src/xbt/xbt_str_test.cpp | 1 - teshsuite/surf/surf_usage/surf_usage.cpp | 1 - 37 files changed, 23 insertions(+), 38 deletions(-) diff --git a/contrib/benchmarking_code_block/bench.h b/contrib/benchmarking_code_block/bench.h index 81b862e753..f458b60612 100644 --- a/contrib/benchmarking_code_block/bench.h +++ b/contrib/benchmarking_code_block/bench.h @@ -61,7 +61,6 @@ static inline void xbt_bench_init(char *tracefile) mybench->output = fopen(tracefile, "a+"); if (mybench->output == NULL) printf("Error while opening the tracefile"); - } /* Initializing StarPU benchmarking */ @@ -72,7 +71,6 @@ static inline void bench_init_starpu(char *tracefile, bench_t *bench) mybench->output = fopen(tracefile, "a+"); if (mybench->output == NULL) printf("Error while opening the tracefile"); - } /* Start benchmarking using macros */ diff --git a/docs/source/tuto_s4u/master-workers-lab1.cpp b/docs/source/tuto_s4u/master-workers-lab1.cpp index b6505bac45..c71c60b08c 100644 --- a/docs/source/tuto_s4u/master-workers-lab1.cpp +++ b/docs/source/tuto_s4u/master-workers-lab1.cpp @@ -64,7 +64,6 @@ static void worker(std::vector args) if (compute_cost > 0) /* If compute_cost is valid, execute a computation of that cost */ simgrid::s4u::this_actor::execute(compute_cost); - } while (compute_cost > 0); /* Stop when receiving an invalid compute_cost */ XBT_INFO("Exiting now."); diff --git a/docs/source/tuto_s4u/master-workers-lab2.cpp b/docs/source/tuto_s4u/master-workers-lab2.cpp index 64176fae17..573c5f3707 100644 --- a/docs/source/tuto_s4u/master-workers-lab2.cpp +++ b/docs/source/tuto_s4u/master-workers-lab2.cpp @@ -29,7 +29,6 @@ static void worker() if (compute_cost > 0) /* If compute_cost is valid, execute a computation of that cost */ simgrid::s4u::this_actor::execute(compute_cost); - } while (compute_cost > 0); /* Stop when receiving an invalid compute_cost */ XBT_INFO("Exiting now."); diff --git a/examples/deprecated/msg/dht-kademlia/node.h b/examples/deprecated/msg/dht-kademlia/node.h index 0a337302df..f67f2cf47e 100644 --- a/examples/deprecated/msg/dht-kademlia/node.h +++ b/examples/deprecated/msg/dht-kademlia/node.h @@ -32,7 +32,6 @@ typedef struct s_node { char mailbox[MAILBOX_NAME_SIZE]; //node mailbox unsigned int find_node_success; //Number of find_node which have succeeded. unsigned int find_node_failed; //Number of find_node which have failed. - } s_node_t; typedef s_node_t *node_t; diff --git a/examples/s4u/actor-lifetime/s4u-actor-lifetime.cpp b/examples/s4u/actor-lifetime/s4u-actor-lifetime.cpp index d53a4ad10a..00c1ea443e 100644 --- a/examples/s4u/actor-lifetime/s4u-actor-lifetime.cpp +++ b/examples/s4u/actor-lifetime/s4u-actor-lifetime.cpp @@ -12,7 +12,6 @@ XBT_LOG_NEW_DEFAULT_CATEGORY(test, "Messages specific for this s4u example"); /* This actor just sleeps until termination */ class sleeper { - public: explicit sleeper(std::vector /*args*/) { diff --git a/examples/s4u/app-bittorrent/s4u-peer.hpp b/examples/s4u/app-bittorrent/s4u-peer.hpp index e828e1d891..9b912bbb94 100644 --- a/examples/s4u/app-bittorrent/s4u-peer.hpp +++ b/examples/s4u/app-bittorrent/s4u-peer.hpp @@ -44,6 +44,7 @@ class Peer { simgrid::s4u::CommPtr comm_received = nullptr; // current comm Message* message = nullptr; // current message being received + public: explicit Peer(std::vector args); Peer(const Peer&) = delete; diff --git a/examples/s4u/app-masterworkers/s4u-app-masterworkers-class.cpp b/examples/s4u/app-masterworkers/s4u-app-masterworkers-class.cpp index cf43661142..d37a880166 100644 --- a/examples/s4u/app-masterworkers/s4u-app-masterworkers-class.cpp +++ b/examples/s4u/app-masterworkers/s4u-app-masterworkers-class.cpp @@ -73,7 +73,6 @@ public: if (compute_cost > 0) /* If compute_cost is valid, execute a computation of that cost */ simgrid::s4u::this_actor::execute(compute_cost); - } while (compute_cost > 0); /* Stop when receiving an invalid compute_cost */ XBT_INFO("Exiting now."); diff --git a/examples/s4u/async-waituntil/s4u-async-waituntil.cpp b/examples/s4u/async-waituntil/s4u-async-waituntil.cpp index 3b2975bafe..ae9039f4fa 100644 --- a/examples/s4u/async-waituntil/s4u-async-waituntil.cpp +++ b/examples/s4u/async-waituntil/s4u-async-waituntil.cpp @@ -28,11 +28,11 @@ static int sender(int argc, char** argv) /* Start dispatching all messages to receivers, in a round robin fashion */ for (int i = 0; i < messages_count; i++) { - std::string mboxName = std::string("receiver-") + std::to_string(i % receivers_count); simgrid::s4u::Mailbox* mbox = simgrid::s4u::Mailbox::by_name(mboxName); std::string msgName = std::string("Message ") + std::to_string(i); std::string* payload = new std::string(msgName); // copy the data we send: + // 'msgName' is not a stable storage location XBT_INFO("Send '%s' to '%s'", msgName.c_str(), mboxName.c_str()); /* Create a communication representing the ongoing communication */ diff --git a/examples/smpi/smpi_s4u_masterslave/masterslave_mailbox_smpi.cpp b/examples/smpi/smpi_s4u_masterslave/masterslave_mailbox_smpi.cpp index 1e583cee66..14d72a3e0d 100644 --- a/examples/smpi/smpi_s4u_masterslave/masterslave_mailbox_smpi.cpp +++ b/examples/smpi/smpi_s4u_masterslave/masterslave_mailbox_smpi.cpp @@ -56,7 +56,6 @@ static void worker(std::vector args) if (compute_cost > 0) /* If compute_cost is valid, execute a computation of that cost */ simgrid::s4u::this_actor::execute(compute_cost); - } while (compute_cost > 0); /* Stop when receiving an invalid compute_cost */ XBT_INFO("Exiting now."); diff --git a/include/simgrid/kernel/routing/NetPoint.hpp b/include/simgrid/kernel/routing/NetPoint.hpp index bddc55681f..39b74e3fb5 100644 --- a/include/simgrid/kernel/routing/NetPoint.hpp +++ b/include/simgrid/kernel/routing/NetPoint.hpp @@ -25,7 +25,6 @@ namespace routing { * @details This represents a position in the network. One can send information between two netpoints */ class NetPoint : public simgrid::xbt::Extendable { - public: enum class Type { Host, Router, NetZone }; diff --git a/include/simgrid/s4u/Activity.hpp b/include/simgrid/s4u/Activity.hpp index 33753de9b6..1757964d85 100644 --- a/include/simgrid/s4u/Activity.hpp +++ b/include/simgrid/s4u/Activity.hpp @@ -115,6 +115,7 @@ public: } friend void intrusive_ptr_add_ref(Activity* a) { a->refcount_.fetch_add(1, std::memory_order_relaxed); } #endif + private: kernel::activity::ActivityImplPtr pimpl_ = nullptr; Activity::State state_ = Activity::State::INITED; diff --git a/include/simgrid/s4u/Storage.hpp b/include/simgrid/s4u/Storage.hpp index 3247dff7f3..9d4c46d893 100644 --- a/include/simgrid/s4u/Storage.hpp +++ b/include/simgrid/s4u/Storage.hpp @@ -38,6 +38,7 @@ public: protected: virtual ~Storage() = default; + public: /** @brief Callback signal fired when a new Storage is created */ static xbt::signal on_creation; diff --git a/include/simgrid/simix/blocking_simcall.hpp b/include/simgrid/simix/blocking_simcall.hpp index 973ead8284..c4d550d25a 100644 --- a/include/simgrid/simix/blocking_simcall.hpp +++ b/include/simgrid/simix/blocking_simcall.hpp @@ -135,6 +135,7 @@ public: }, nullptr); } + private: // We wrap an event-based kernel future: simgrid::kernel::Future future_; diff --git a/include/xbt/future.hpp b/include/xbt/future.hpp index 1dccc74d5d..ac75bf6747 100644 --- a/include/xbt/future.hpp +++ b/include/xbt/future.hpp @@ -70,6 +70,7 @@ public: throw std::future_error(std::future_errc::no_state); } } + private: boost::variant value_; }; diff --git a/src/bindings/lua/lua_utils.cpp b/src/bindings/lua/lua_utils.cpp index 1bb9e9c8ad..952f4de0a6 100644 --- a/src/bindings/lua/lua_utils.cpp +++ b/src/bindings/lua/lua_utils.cpp @@ -26,11 +26,9 @@ */ const char* sglua_tostring(lua_State* L, int index) { - static char buff[64]; switch (lua_type(L, index)) { - case LUA_TNIL: snprintf(buff, 4, "nil"); break; @@ -87,7 +85,6 @@ const char* sglua_tostring(lua_State* L, int index) */ const char* sglua_keyvalue_tostring(lua_State* L, int key_index, int value_index) { - static char buff[64]; /* value_tostring also always returns the same pointer */ int len = snprintf(buff, 63, "[%s] -> ", sglua_tostring(L, key_index)); diff --git a/src/bindings/lua/simgrid_lua.cpp b/src/bindings/lua/simgrid_lua.cpp index e96c07918f..7a0f6f9be4 100644 --- a/src/bindings/lua/simgrid_lua.cpp +++ b/src/bindings/lua/simgrid_lua.cpp @@ -26,8 +26,8 @@ extern "C" int luaopen_simgrid(lua_State* L); * * - Argument 1 (string): the text to print */ -static int debug(lua_State* L) { - +static int debug(lua_State* L) +{ const char* str = luaL_checkstring(L, 1); XBT_DEBUG("%s", str); return 0; diff --git a/src/instr/instr_paje_containers.hpp b/src/instr/instr_paje_containers.hpp index 4a76b0cb6d..952f7d9e43 100644 --- a/src/instr/instr_paje_containers.hpp +++ b/src/instr/instr_paje_containers.hpp @@ -19,6 +19,7 @@ class VariableType; class Container { long long int id_; std::string name_; /* Unique name of this container */ + public: Container(const std::string& name, const std::string& type_name, Container* father); Container(const Container&) = delete; diff --git a/src/kernel/context/ContextBoost.cpp b/src/kernel/context/ContextBoost.cpp index 9d275c2d3c..d79a549d5e 100644 --- a/src/kernel/context/ContextBoost.cpp +++ b/src/kernel/context/ContextBoost.cpp @@ -24,7 +24,6 @@ BoostContext* BoostContextFactory::create_context(std::function&& code, BoostContext::BoostContext(std::function&& code, actor::ActorImpl* actor, SwappedContextFactory* factory) : SwappedContext(std::move(code), actor, factory) { - /* if the user provided a function for the process then use it, otherwise it is the context for maestro */ if (has_code()) { #if BOOST_VERSION < 106100 diff --git a/src/kernel/resource/profile/Profile_test.cpp b/src/kernel/resource/profile/Profile_test.cpp index c60517a6fb..5243791e2f 100644 --- a/src/kernel/resource/profile/Profile_test.cpp +++ b/src/kernel/resource/profile/Profile_test.cpp @@ -65,7 +65,6 @@ static std::vector trace2vector(const char TEST_CASE("kernel::profile: Resource profiles, defining the external load", "kernel::profile") { - SECTION("No event, no loop") { std::vector got = trace2vector(""); @@ -96,7 +95,6 @@ TEST_CASE("kernel::profile: Resource profiles, defining the external load", "ker SECTION("Three events, no loop") { - std::vector got = trace2vector("3.0 1.0\n" "5.0 2.0\n" "9.0 3.0\n"); diff --git a/src/kernel/routing/ClusterZone.cpp b/src/kernel/routing/ClusterZone.cpp index 860786e77c..21c4b65647 100644 --- a/src/kernel/routing/ClusterZone.cpp +++ b/src/kernel/routing/ClusterZone.cpp @@ -62,9 +62,9 @@ void ClusterZone::get_local_route(NetPoint* src, NetPoint* dst, RouteCreationArg } if (not dst->is_router()) { // No specific link for router - std::pair info = private_links_.at(node_pos_with_loopback_limiter(dst->id())); + if (info.second) { // link down route->link_list.push_back(info.second); if (lat) diff --git a/src/mc/AddressSpace.hpp b/src/mc/AddressSpace.hpp index 8701fa2597..6a5392c902 100644 --- a/src/mc/AddressSpace.hpp +++ b/src/mc/AddressSpace.hpp @@ -21,6 +21,7 @@ namespace mc { class ReadOptions { std::uint32_t value_ = 0; constexpr explicit ReadOptions(std::uint32_t value) : value_(value) {} + public: constexpr ReadOptions() {} @@ -132,7 +133,6 @@ public: this->read_bytes(&res[0], len, address); return res; } - }; } diff --git a/src/mc/ModelChecker.hpp b/src/mc/ModelChecker.hpp index 53ff36b222..0f33b43f66 100644 --- a/src/mc/ModelChecker.hpp +++ b/src/mc/ModelChecker.hpp @@ -30,8 +30,8 @@ class ModelChecker { PageStore page_store_{500}; std::unique_ptr process_; Checker* checker_ = nullptr; -public: +public: ModelChecker(ModelChecker const&) = delete; ModelChecker& operator=(ModelChecker const&) = delete; explicit ModelChecker(std::unique_ptr process); diff --git a/src/mc/checker/Checker.hpp b/src/mc/checker/Checker.hpp index 8527b76578..c9ff8336ef 100644 --- a/src/mc/checker/Checker.hpp +++ b/src/mc/checker/Checker.hpp @@ -29,6 +29,7 @@ namespace mc { // abstract class Checker { Session* session_; + public: explicit Checker(Session& session); diff --git a/src/mc/checker/CommunicationDeterminismChecker.hpp b/src/mc/checker/CommunicationDeterminismChecker.hpp index a9d50e6d12..1701a284ab 100644 --- a/src/mc/checker/CommunicationDeterminismChecker.hpp +++ b/src/mc/checker/CommunicationDeterminismChecker.hpp @@ -30,6 +30,7 @@ private: void log_state() override; void deterministic_comm_pattern(int process, const simgrid::mc::PatternCommunication* comm, int backtracking); void restoreState(); + public: // These are used by functions which should be moved in CommunicationDeterminismChecker: void get_comm_pattern(smx_simcall_t request, e_mc_call_type_t call_type, int backtracking); diff --git a/src/mc/mc_comm_pattern.cpp b/src/mc/mc_comm_pattern.cpp index db46857d9c..9a3d3b5b34 100644 --- a/src/mc/mc_comm_pattern.cpp +++ b/src/mc/mc_comm_pattern.cpp @@ -78,5 +78,4 @@ void MC_handle_comm_pattern(e_mc_call_type_t call_type, smx_simcall_t req, int v default: xbt_die("Unexpected call type %i", (int)call_type); } - } diff --git a/src/mc/mc_global.cpp b/src/mc/mc_global.cpp index 2bfdb0cff4..4d6c6e351f 100644 --- a/src/mc/mc_global.cpp +++ b/src/mc/mc_global.cpp @@ -69,7 +69,6 @@ void MC_init_dot_output() fprintf(dot_output, "digraph graphname{\n fixedsize=true; rankdir=TB; ranksep=.25; edge [fontsize=12]; node [fontsize=10, shape=circle,width=.5 ]; graph [resolution=20, fontsize=10];\n"); - } /******************************* Core of MC *******************************/ diff --git a/src/mc/mc_record.cpp b/src/mc/mc_record.cpp index bdd03aaa25..78e34fee4a 100644 --- a/src/mc/mc_record.cpp +++ b/src/mc/mc_record.cpp @@ -63,9 +63,9 @@ RecordTrace parseRecordTrace(const char* data) const char* current = data; while (*current) { - simgrid::mc::Transition item; int count = sscanf(current, "%d/%d", &item.pid_, &item.argument_); + if(count != 2 && count != 1) throw std::invalid_argument("Could not parse record path"); res.push_back(item); diff --git a/src/mc/mc_smx.cpp b/src/mc/mc_smx.cpp index af66ee9db3..f72a03b50c 100644 --- a/src/mc/mc_smx.cpp +++ b/src/mc/mc_smx.cpp @@ -45,8 +45,8 @@ static void MC_process_refresh_simix_actor_dynar(const simgrid::mc::RemoteClient // Load each element of the vector from the MCed process: for (unsigned int i = 0; i < dynar.used; ++i) { - simgrid::mc::ActorInformation info; + info.address = simgrid::mc::RemotePtr(data[i]); info.hostname = nullptr; process->read_bytes(&info.copy, sizeof(info.copy), remote(data[i])); diff --git a/src/mc/sosp/ChunkedData.cpp b/src/mc/sosp/ChunkedData.cpp index ef35495c2c..f93c4e2e07 100644 --- a/src/mc/sosp/ChunkedData.cpp +++ b/src/mc/sosp/ChunkedData.cpp @@ -22,7 +22,6 @@ ChunkedData::ChunkedData(PageStore& store, const AddressSpace& as, RemotePtr buffer(xbt_pagesize); for (size_t i = 0; i != page_count; ++i) { - RemotePtr page = remote((void*)simgrid::mc::mmu::join(i, addr.address())); xbt_assert(simgrid::mc::mmu::split(page.address()).second == 0, "Not at the beginning of a page"); diff --git a/src/mc/sosp/Snapshot_test.cpp b/src/mc/sosp/Snapshot_test.cpp index 942e88fa79..36a7298b5b 100644 --- a/src/mc/sosp/Snapshot_test.cpp +++ b/src/mc/sosp/Snapshot_test.cpp @@ -127,7 +127,6 @@ void snap_test_helper::read_region_parts() void snap_test_helper::compare_whole_region() { for (int n = 1; n != 32; ++n) { - prologue_return ret = prologue(n); INFO("Unexpected match in MC_snapshot_region_memcmp() with previous snapshot"); diff --git a/src/msg/msg_global.cpp b/src/msg/msg_global.cpp index 6d4b18eca3..c5004ba48a 100644 --- a/src/msg/msg_global.cpp +++ b/src/msg/msg_global.cpp @@ -26,8 +26,8 @@ static void MSG_exit(); * @brief Initialize MSG with less verifications * You should use the MSG_init() function instead. Failing to do so may turn into PEBKAC some day. You've been warned. */ -void MSG_init_nocheck(int *argc, char **argv) { - +void MSG_init_nocheck(int* argc, char** argv) +{ TRACE_global_init(); if (not msg_global) { @@ -58,12 +58,14 @@ void MSG_init_nocheck(int *argc, char **argv) { atexit(MSG_exit); } -void MSG_config(const char *key, const char *value){ +void MSG_config(const char* key, const char* value) +{ xbt_assert(msg_global,"ERROR: Please call MSG_init() before using MSG_config()"); simgrid::config::set_as_string(key, value); } -static void MSG_exit() { +static void MSG_exit() +{ delete msg_global; msg_global = nullptr; } diff --git a/src/surf/network_ib.hpp b/src/surf/network_ib.hpp index 8df5bc0135..4b14ff2b60 100644 --- a/src/surf/network_ib.hpp +++ b/src/surf/network_ib.hpp @@ -54,7 +54,6 @@ public: std::unordered_map active_nodes; std::unordered_map> active_comms; - }; } // namespace resource } // namespace kernel diff --git a/src/surf/ns3/ns3_simulator.cpp b/src/surf/ns3/ns3_simulator.cpp index 180d0a4d68..ebff56b10c 100644 --- a/src/surf/ns3/ns3_simulator.cpp +++ b/src/surf/ns3/ns3_simulator.cpp @@ -67,9 +67,9 @@ static void send_cb(ns3::Ptr sock, uint32_t txSpace) /* While not all is buffered and there remain space in the buffers */ while (flow->buffered_bytes_ < flow->total_bytes_ && sock->GetTxAvailable() > 0) { - // Send at most 1040 bytes (data size in a TCP packet), as ns-3 seems to not split correctly by itself uint32_t toWrite = std::min({flow->remaining_, sock->GetTxAvailable(), std::uint32_t(1040)}); + if (toWrite == 0) { // buffer full XBT_DEBUG("%f: buffer full on flow %p (still %u to go)", ns3::Simulator::Now().GetSeconds(), flow, flow->remaining_); diff --git a/src/xbt/config_test.cpp b/src/xbt/config_test.cpp index e9c2856d21..b480f5b111 100644 --- a/src/xbt/config_test.cpp +++ b/src/xbt/config_test.cpp @@ -25,7 +25,6 @@ static void make_set() TEST_CASE("xbt::config: Configuration support", "config") { - SECTION("Alloc and free a config set") { auto temp = simgrid_config; diff --git a/src/xbt/mmalloc/mmprivate.h b/src/xbt/mmalloc/mmprivate.h index 63ccf0fb99..2d7af4a2b6 100644 --- a/src/xbt/mmalloc/mmprivate.h +++ b/src/xbt/mmalloc/mmprivate.h @@ -176,7 +176,6 @@ typedef struct { * if such a file exists. * */ struct mdesc { - /** @brief Mutex locking the access to the heap */ pthread_mutex_t mutex; @@ -261,7 +260,6 @@ struct mdesc { /* @brief Instrumentation */ struct mstats heapstats; - }; /* Bits to look at in the malloc descriptor flags word */ diff --git a/src/xbt/xbt_str_test.cpp b/src/xbt/xbt_str_test.cpp index e7d70b0f44..6a3bc606fd 100644 --- a/src/xbt/xbt_str_test.cpp +++ b/src/xbt/xbt_str_test.cpp @@ -43,7 +43,6 @@ template void test_parse_ok(F function, const std::stri TEST_CASE("xbt::str: String Handling", "xbt_str") { - SECTION("Test the function xbt_str_split_quoted") { test_split_quoted("Empty", "", {}); diff --git a/teshsuite/surf/surf_usage/surf_usage.cpp b/teshsuite/surf/surf_usage/surf_usage.cpp index e1b96f9549..35009f9abd 100644 --- a/teshsuite/surf/surf_usage/surf_usage.cpp +++ b/teshsuite/surf/surf_usage/surf_usage.cpp @@ -99,7 +99,6 @@ int main(int argc, char **argv) XBT_DEBUG("\t * Done : %p", &action); action.unref(); } - } while ( (surf_network_model->get_started_action_set()->size() || surf_cpu_model_pm->get_started_action_set()->size()) && surf_solve(-1.0) >= 0.0); -- 2.20.1