diff options
| author | Benoit Foucher <benoit@zeroc.com> | 2017-10-12 08:31:21 +0200 |
|---|---|---|
| committer | Benoit Foucher <benoit@zeroc.com> | 2017-10-12 08:31:21 +0200 |
| commit | c0304fed7365d5086f9f08c145285a65375acb02 (patch) | |
| tree | 04ac95367c7dabd6449b67ce7a50831a94a3277b /cpp/src | |
| parent | JavaScript testsuite improvements and simplifications (diff) | |
| download | ice-c0304fed7365d5086f9f08c145285a65375acb02.tar.bz2 ice-c0304fed7365d5086f9f08c145285a65375acb02.tar.xz ice-c0304fed7365d5086f9f08c145285a65375acb02.zip | |
Fixed generated code warnings showing up with Objective-C and Xcode 9.0 (ICE-8540)
Diffstat (limited to 'cpp/src')
| -rw-r--r-- | cpp/src/slice2objc/Gen.cpp | 4 | ||||
| -rw-r--r-- | cpp/src/slice2objc/ObjCUtil.cpp | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/cpp/src/slice2objc/Gen.cpp b/cpp/src/slice2objc/Gen.cpp index c2ff2e2bca9..1c4b7e58943 100644 --- a/cpp/src/slice2objc/Gen.cpp +++ b/cpp/src/slice2objc/Gen.cpp @@ -449,7 +449,7 @@ Slice::ObjCVisitor::getParams(const OperationPtr& op, bool internal) const string typeString; if((*q)->isOutParam()) { - typeString = outTypeToString(type, (*q)->optional(), false, true); + typeString = outTypeToString(type, (*q)->optional(), true, true); } else { @@ -524,7 +524,7 @@ Slice::ObjCVisitor::getUnmarshalParams(const OperationPtr& op, bool internal) co { result += " " + getParamId(*q); } - result += ":(" + outTypeToString(type, (*q)->optional(), false, true) + ")" + getParamName(*q, internal); + result += ":(" + outTypeToString(type, (*q)->optional(), true, true) + ")" + getParamName(*q, internal); } } return result; diff --git a/cpp/src/slice2objc/ObjCUtil.cpp b/cpp/src/slice2objc/ObjCUtil.cpp index 253519c837a..d5bf0e6d9c3 100644 --- a/cpp/src/slice2objc/ObjCUtil.cpp +++ b/cpp/src/slice2objc/ObjCUtil.cpp @@ -406,7 +406,7 @@ Slice::ObjCGenerator::outTypeToString(const TypePtr& type, bool optional, bool a s += "*"; } } - if(autoreleasing && !isValueType(type)) + if(autoreleasing && (!isValueType(type) || optional)) { s += " ICE_AUTORELEASING_QUALIFIER"; } |