From 1249a2a550c7a297823f6fa81eb3094b01863eab Mon Sep 17 00:00:00 2001 From: Jethro Grassie Date: Wed, 16 Aug 2017 09:34:32 -0400 Subject: [PATCH] Construct on first use for completion_commands --- contrib/epee/include/readline_buffer.h | 14 +++----------- contrib/epee/src/readline_buffer.cpp | 24 ++++++++++++++++++++---- 2 files changed, 23 insertions(+), 15 deletions(-) diff --git a/contrib/epee/include/readline_buffer.h b/contrib/epee/include/readline_buffer.h index 8dd082a7..28a15341 100644 --- a/contrib/epee/include/readline_buffer.h +++ b/contrib/epee/include/readline_buffer.h @@ -21,23 +21,15 @@ namespace rdln } void get_line(std::string& line) const; void set_prompt(const std::string& prompt); - static void add_completion(const std::string& command) - { - if(std::find(completion_commands.begin(), completion_commands.end(), command) != completion_commands.end()) - return; - completion_commands.push_back(command); - } - static const std::vector& get_completions() - { - return completion_commands; - } + static void add_completion(const std::string& command); + static const std::vector& get_completions(); protected: virtual int sync(); private: std::streambuf* m_cout_buf; - static std::vector completion_commands; + static std::vector& completion_commands(); }; class suspend_readline diff --git a/contrib/epee/src/readline_buffer.cpp b/contrib/epee/src/readline_buffer.cpp index d4051bcb..42b47405 100644 --- a/contrib/epee/src/readline_buffer.cpp +++ b/contrib/epee/src/readline_buffer.cpp @@ -14,10 +14,8 @@ static void remove_line_handler(); static std::string last_line; static std::string last_prompt; -std::mutex line_mutex, sync_mutex, process_mutex; -std::condition_variable have_line; - -std::vector rdln::readline_buffer::completion_commands = {"exit"}; +static std::mutex line_mutex, sync_mutex, process_mutex; +static std::condition_variable have_line; namespace { @@ -43,6 +41,12 @@ rdln::suspend_readline::~suspend_readline() m_buffer->start(); } +std::vector& rdln::readline_buffer::completion_commands() +{ + static std::vector commands = {"exit"}; + return commands; +} + rdln::readline_buffer::readline_buffer() : std::stringbuf(), m_cout_buf(NULL) { @@ -88,6 +92,18 @@ void rdln::readline_buffer::set_prompt(const std::string& prompt) rl_redisplay(); } +void rdln::readline_buffer::add_completion(const std::string& command) +{ + if(std::find(completion_commands().begin(), completion_commands().end(), command) != completion_commands().end()) + return; + completion_commands().push_back(command); +} + +const std::vector& rdln::readline_buffer::get_completions() +{ + return completion_commands(); +} + int rdln::readline_buffer::process() { process_mutex.lock();