diff options
author | Andrey Lihatskiy <alihatskiy@productengine.com> | 2024-04-29 07:43:28 +0300 |
---|---|---|
committer | Andrey Lihatskiy <alihatskiy@productengine.com> | 2024-04-29 07:56:09 +0300 |
commit | 1b68f71348ecf3983b76b40d7940da8377f049b7 (patch) | |
tree | 2974eddaef130a067c26033d60a59fc790365b3d /indra/newview/llexternaleditor.cpp | |
parent | af4ea94efc1999f3b19fd8d643d0331f0b77e265 (diff) |
#824 Process source files in bulk: replace tabs with spaces, convert CRLF to LF, and trim trailing whitespaces as needed
Diffstat (limited to 'indra/newview/llexternaleditor.cpp')
-rw-r--r-- | indra/newview/llexternaleditor.cpp | 278 |
1 files changed, 139 insertions, 139 deletions
diff --git a/indra/newview/llexternaleditor.cpp b/indra/newview/llexternaleditor.cpp index b0d88159c1..f96dd6fe67 100644 --- a/indra/newview/llexternaleditor.cpp +++ b/indra/newview/llexternaleditor.cpp @@ -1,25 +1,25 @@ -/** +/** * @file llexternaleditor.cpp * @brief A convenient class to run external editor. * * $LicenseInfo:firstyear=2010&license=viewerlgpl$ * Second Life Viewer Source Code * Copyright (C) 2010, Linden Research, Inc. - * + * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public * License as published by the Free Software Foundation; * version 2.1 of the License only. - * + * * This library is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * Lesser General Public License for more details. - * + * * You should have received a copy of the GNU Lesser General Public * License along with this library; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA - * + * * Linden Research, Inc., 945 Battery Street, San Francisco, CA 94111 USA * $/LicenseInfo$ */ @@ -41,82 +41,82 @@ const std::string LLExternalEditor::sSetting = "ExternalEditor"; LLExternalEditor::EErrorCode LLExternalEditor::setCommand(const std::string& env_var, const std::string& override) { - std::string cmd = findCommand(env_var, override); - if (cmd.empty()) - { - LL_WARNS() << "Editor command is empty or not set" << LL_ENDL; - return EC_NOT_SPECIFIED; - } - - string_vec_t tokens; - tokenize(tokens, cmd); - - // Check executable for existence. - std::string bin_path = tokens[0]; - if (!LLFile::isfile(bin_path)) - { - LL_WARNS() << "Editor binary [" << bin_path << "] not found" << LL_ENDL; - return EC_BINARY_NOT_FOUND; - } - - // Save command. - mProcessParams = LLProcess::Params(); - mProcessParams.executable = bin_path; - for (size_t i = 1; i < tokens.size(); ++i) - { - mProcessParams.args.add(tokens[i]); - } - - // Add the filename marker if missing. - if (cmd.find(sFilenameMarker) == std::string::npos) - { - mProcessParams.args.add(sFilenameMarker); - LL_INFOS() << "Adding the filename marker (" << sFilenameMarker << ")" << LL_ENDL; - } - - LL_INFOS() << "Setting command [" << mProcessParams << "]" << LL_ENDL; - - return EC_SUCCESS; + std::string cmd = findCommand(env_var, override); + if (cmd.empty()) + { + LL_WARNS() << "Editor command is empty or not set" << LL_ENDL; + return EC_NOT_SPECIFIED; + } + + string_vec_t tokens; + tokenize(tokens, cmd); + + // Check executable for existence. + std::string bin_path = tokens[0]; + if (!LLFile::isfile(bin_path)) + { + LL_WARNS() << "Editor binary [" << bin_path << "] not found" << LL_ENDL; + return EC_BINARY_NOT_FOUND; + } + + // Save command. + mProcessParams = LLProcess::Params(); + mProcessParams.executable = bin_path; + for (size_t i = 1; i < tokens.size(); ++i) + { + mProcessParams.args.add(tokens[i]); + } + + // Add the filename marker if missing. + if (cmd.find(sFilenameMarker) == std::string::npos) + { + mProcessParams.args.add(sFilenameMarker); + LL_INFOS() << "Adding the filename marker (" << sFilenameMarker << ")" << LL_ENDL; + } + + LL_INFOS() << "Setting command [" << mProcessParams << "]" << LL_ENDL; + + return EC_SUCCESS; } LLExternalEditor::EErrorCode LLExternalEditor::run(const std::string& file_path) { - if (std::string(mProcessParams.executable).empty() || mProcessParams.args.empty()) - { - LL_WARNS() << "Editor command not set" << LL_ENDL; - return EC_NOT_SPECIFIED; - } - - // Copy params block so we can replace sFilenameMarker - LLProcess::Params params; - params.executable = mProcessParams.executable; - - // Substitute the filename marker in the command with the actual passed file name. - for (const std::string& arg : mProcessParams.args) - { - std::string fixed(arg); - LLStringUtil::replaceString(fixed, sFilenameMarker, file_path); - params.args.add(fixed); - } - - // Run the editor. Prevent killing the process in destructor. - params.autokill = false; - return LLProcess::create(params) ? EC_SUCCESS : EC_FAILED_TO_RUN; + if (std::string(mProcessParams.executable).empty() || mProcessParams.args.empty()) + { + LL_WARNS() << "Editor command not set" << LL_ENDL; + return EC_NOT_SPECIFIED; + } + + // Copy params block so we can replace sFilenameMarker + LLProcess::Params params; + params.executable = mProcessParams.executable; + + // Substitute the filename marker in the command with the actual passed file name. + for (const std::string& arg : mProcessParams.args) + { + std::string fixed(arg); + LLStringUtil::replaceString(fixed, sFilenameMarker, file_path); + params.args.add(fixed); + } + + // Run the editor. Prevent killing the process in destructor. + params.autokill = false; + return LLProcess::create(params) ? EC_SUCCESS : EC_FAILED_TO_RUN; } // static std::string LLExternalEditor::getErrorMessage(EErrorCode code) { - switch (code) - { - case EC_SUCCESS: return LLTrans::getString("ok"); - case EC_NOT_SPECIFIED: return LLTrans::getString("ExternalEditorNotSet"); - case EC_PARSE_ERROR: return LLTrans::getString("ExternalEditorCommandParseError"); - case EC_BINARY_NOT_FOUND: return LLTrans::getString("ExternalEditorNotFound"); - case EC_FAILED_TO_RUN: return LLTrans::getString("ExternalEditorFailedToRun"); - } - - return LLTrans::getString("Unknown"); + switch (code) + { + case EC_SUCCESS: return LLTrans::getString("ok"); + case EC_NOT_SPECIFIED: return LLTrans::getString("ExternalEditorNotSet"); + case EC_PARSE_ERROR: return LLTrans::getString("ExternalEditorCommandParseError"); + case EC_BINARY_NOT_FOUND: return LLTrans::getString("ExternalEditorNotFound"); + case EC_FAILED_TO_RUN: return LLTrans::getString("ExternalEditorFailedToRun"); + } + + return LLTrans::getString("Unknown"); } // TODO: @@ -128,76 +128,76 @@ std::string LLExternalEditor::getErrorMessage(EErrorCode code) // static size_t LLExternalEditor::tokenize(string_vec_t& tokens, const std::string& str) { - tokens.clear(); - - // Split the argument string into separate strings for each argument - typedef boost::tokenizer< boost::char_separator<char> > tokenizer; - boost::char_separator<char> sep("", "\" ", boost::drop_empty_tokens); - - tokenizer tokens_list(str, sep); - tokenizer::iterator token_iter; - BOOL inside_quotes = FALSE; - BOOL last_was_space = FALSE; - for (token_iter = tokens_list.begin(); token_iter != tokens_list.end(); ++token_iter) - { - if (!strncmp("\"",(*token_iter).c_str(),2)) - { - inside_quotes = !inside_quotes; - } - else if (!strncmp(" ",(*token_iter).c_str(),2)) - { - if(inside_quotes) - { - tokens.back().append(std::string(" ")); - last_was_space = TRUE; - } - } - else - { - std::string to_push = *token_iter; - if (last_was_space) - { - tokens.back().append(to_push); - last_was_space = FALSE; - } - else - { - tokens.push_back(to_push); - } - } - } - - return tokens.size(); + tokens.clear(); + + // Split the argument string into separate strings for each argument + typedef boost::tokenizer< boost::char_separator<char> > tokenizer; + boost::char_separator<char> sep("", "\" ", boost::drop_empty_tokens); + + tokenizer tokens_list(str, sep); + tokenizer::iterator token_iter; + BOOL inside_quotes = FALSE; + BOOL last_was_space = FALSE; + for (token_iter = tokens_list.begin(); token_iter != tokens_list.end(); ++token_iter) + { + if (!strncmp("\"",(*token_iter).c_str(),2)) + { + inside_quotes = !inside_quotes; + } + else if (!strncmp(" ",(*token_iter).c_str(),2)) + { + if(inside_quotes) + { + tokens.back().append(std::string(" ")); + last_was_space = TRUE; + } + } + else + { + std::string to_push = *token_iter; + if (last_was_space) + { + tokens.back().append(to_push); + last_was_space = FALSE; + } + else + { + tokens.push_back(to_push); + } + } + } + + return tokens.size(); } // static std::string LLExternalEditor::findCommand( - const std::string& env_var, - const std::string& override) + const std::string& env_var, + const std::string& override) { - std::string cmd; - - // Get executable path. - if (!override.empty()) // try the supplied override first - { - cmd = override; - LL_INFOS() << "Using override" << LL_ENDL; - } - else if (!LLUI::getInstance()->mSettingGroups["config"]->getString(sSetting).empty()) - { - cmd = LLUI::getInstance()->mSettingGroups["config"]->getString(sSetting); - LL_INFOS() << "Using setting" << LL_ENDL; - } - else // otherwise use the path specified by the environment variable - { - auto env_var_val(LLStringUtil::getoptenv(env_var)); - if (env_var_val) - { - cmd = *env_var_val; - LL_INFOS() << "Using env var " << env_var << LL_ENDL; - } - } - - LL_INFOS() << "Found command [" << cmd << "]" << LL_ENDL; - return cmd; + std::string cmd; + + // Get executable path. + if (!override.empty()) // try the supplied override first + { + cmd = override; + LL_INFOS() << "Using override" << LL_ENDL; + } + else if (!LLUI::getInstance()->mSettingGroups["config"]->getString(sSetting).empty()) + { + cmd = LLUI::getInstance()->mSettingGroups["config"]->getString(sSetting); + LL_INFOS() << "Using setting" << LL_ENDL; + } + else // otherwise use the path specified by the environment variable + { + auto env_var_val(LLStringUtil::getoptenv(env_var)); + if (env_var_val) + { + cmd = *env_var_val; + LL_INFOS() << "Using env var " << env_var << LL_ENDL; + } + } + + LL_INFOS() << "Found command [" << cmd << "]" << LL_ENDL; + return cmd; } |