Swiftdiff options
| author | Kevin Smith <git@kismith.co.uk> | 2017-06-20 20:15:25 (GMT) |
|---|---|---|
| committer | Kevin Smith <kevin.smith@isode.com> | 2017-06-21 08:29:48 (GMT) |
| commit | 83fb5ae3da2e5e690a3eafae99f6cd0cb59a2496 (patch) | |
| tree | b2924f4079410642885681d53b1295e5d6125347 /Swift/Controllers | |
| parent | a507a88a189bb603c9f2d686c9c8dafca49c053d (diff) | |
| download | swift-83fb5ae3da2e5e690a3eafae99f6cd0cb59a2496.zip swift-83fb5ae3da2e5e690a3eafae99f6cd0cb59a2496.tar.bz2 | |
Fix compiler warnings
Both catching polymorphic base classes by value and explicitly moving things
preventing return value optimization.
Test-Information:
Compiles on macOS with latest xcode with allow_warnings=False and unit
tests pass
Change-Id: Ibdc67f4f2019f85f14635df2a3ff06bddbb8e2b6
Diffstat (limited to 'Swift/Controllers')
| -rw-r--r-- | Swift/Controllers/Chat/ChatMessageParser.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Swift/Controllers/Chat/ChatMessageParser.cpp b/Swift/Controllers/Chat/ChatMessageParser.cpp index 1a822a1..31be451 100644 --- a/Swift/Controllers/Chat/ChatMessageParser.cpp +++ b/Swift/Controllers/Chat/ChatMessageParser.cpp @@ -132,7 +132,7 @@ namespace Swift { } } - catch (std::runtime_error) { + catch (const std::runtime_error&) { /* Basically too expensive to compute the regex results and it gave up, so pass through as text */ newMessage.append(part); } @@ -185,7 +185,7 @@ namespace Swift { resultMessage.append(std::make_shared<ChatWindow::ChatTextMessagePart>(std::string(start, text.end()))); } } - catch (std::runtime_error) { + catch (const std::runtime_error&) { /* Basically too expensive to compute the regex results and it gave up, so pass through as text */ resultMessage.append(part); } |