diff options
author | 2016-01-10 16:36:18 -0500 | |
---|---|---|
committer | 2016-01-10 16:36:18 -0500 | |
commit | 115db95642ad6d861aad63775b92350e0491f2dd (patch) | |
tree | 8e7538ca804610335bff1927dc3b4d974b4c06da /src/lib/formats/cbm_crt.cpp | |
parent | 2a2a4a3c6e255c479d5072d02c2a40949f611ea6 (diff) |
Return std::string objects by value rather than pass by reference
- strprintf is unaltered, but strformat now takes one fewer argument
- state_string_export still fills a buffer, but has been made const
- get_default_card_software now takes no arguments but returns a string
Diffstat (limited to 'src/lib/formats/cbm_crt.cpp')
-rw-r--r-- | src/lib/formats/cbm_crt.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/lib/formats/cbm_crt.cpp b/src/lib/formats/cbm_crt.cpp index d169d0c2fd7..6484dbf108c 100644 --- a/src/lib/formats/cbm_crt.cpp +++ b/src/lib/formats/cbm_crt.cpp @@ -122,7 +122,7 @@ static const char * CRT_C64_SLOT_NAMES[_CRT_C64_COUNT] = // cbm_crt_get_card - get slot interface card //------------------------------------------------- -void cbm_crt_get_card(std::string &result, core_file *file) +std::string cbm_crt_get_card(core_file *file) { // read the header cbm_crt_header header; @@ -132,11 +132,10 @@ void cbm_crt_get_card(std::string &result, core_file *file) { UINT16 hardware = pick_integer_be(header.hardware, 0, 2); - result.assign(CRT_C64_SLOT_NAMES[hardware]); - return; + return std::string(CRT_C64_SLOT_NAMES[hardware]); } - result.clear(); + return std::string(); } |