Skip to content
Snippets Groups Projects
Commit b91eadf3 authored by Nick Draper's avatar Nick Draper
Browse files

Fix linux nonly warnings

re #12064
parent 12e5aa38
No related branches found
No related tags found
No related merge requests found
......@@ -175,7 +175,7 @@ bool CheckMantidVersion::isVersionMoreRecent(
auto localVersionParts = splitVersionString(localVersion);
auto gitHubVersionParts = splitVersionString(gitHubVersion);
for (int i = 0; i < gitHubVersionParts.size(); i++) {
for (size_t i = 0; i < gitHubVersionParts.size(); i++) {
// sanity check
if (i >= localVersionParts.size()) {
// ran out of items to compare
......
......@@ -14,14 +14,15 @@ namespace {
*/
class MockedCheckMantidVersion : public CheckMantidVersion {
public:
MockedCheckMantidVersion(std::string currentVersion, std::string gitHubVersion) : CurrentVersion(currentVersion),
GitHubVersion(gitHubVersion), CheckMantidVersion() {}
MockedCheckMantidVersion(std::string currentVersion, std::string gitHubVersion) : CheckMantidVersion(), CurrentVersion(currentVersion),
GitHubVersion(gitHubVersion){}
std::string CurrentVersion;
std::string GitHubVersion;
private:
virtual std::string getVersionsFromGitHub(const std::string &url) {
std::string outputString;
//the initial assignment of the value to url is just to suppress a compiler warning
std::string outputString(url);
outputString ="{\n"
" \"url\": \"https://api.github.com/repos/mantidproject/mantid/releases/1308203\",\n"
" \"assets_url\": \"https://api.github.com/repos/mantidproject/mantid/releases/1308203/assets\",\n"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment