From d79b96cf6564187c96f5a1451ca98e2c93adee77 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Fri, 12 Dec 2014 17:00:50 +0100 Subject: css::uno::Sequence ctor with initializer_list (though LIBO_INTERNAL_ONLY, as it needs C++11, so cannot in general be used in URE client code; I think it's better to not offer it outside LO at all, than based on a feature-check macro, and thus catch accidental misuses of it via CppunitTest_odk_checkapi) ...plus adapting binaryurp/ to use the new feature Change-Id: I9a88a0e9eac5daf72896470e8b6a1deb1a6fc88f --- binaryurp/qa/test-unmarshal.cxx | 42 +++++++++++++++----------------------- binaryurp/source/bridgefactory.cxx | 3 +-- 2 files changed, 18 insertions(+), 27 deletions(-) (limited to 'binaryurp') diff --git a/binaryurp/qa/test-unmarshal.cxx b/binaryurp/qa/test-unmarshal.cxx index 759b120a2d20..13ab9f6f99e7 100644 --- a/binaryurp/qa/test-unmarshal.cxx +++ b/binaryurp/qa/test-unmarshal.cxx @@ -50,20 +50,15 @@ private: void Test::testTypeOfBooleanSequence() { binaryurp::ReaderState state; - css::uno::Sequence< sal_Int8 > buf(13); - buf[0] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(20 | 0x80)); // sequence type | cache flag - buf[1] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(binaryurp::cache::ignore >> 8)); - buf[2] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(binaryurp::cache::ignore & 0xFF)); - buf[3] = RTL_CONSTASCII_LENGTH("[]boolean"); - buf[4] = '['; - buf[5] = ']'; - buf[6] = 'b'; - buf[7] = 'o'; - buf[8] = 'o'; - buf[9] = 'l'; - buf[10] = 'e'; - buf[11] = 'a'; - buf[12] = 'n'; + css::uno::Sequence buf{ + static_cast(static_cast(20 | 0x80)), + // sequence type | cache flag + static_cast( + static_cast(binaryurp::cache::ignore >> 8)), + static_cast( + static_cast(binaryurp::cache::ignore & 0xFF)), + RTL_CONSTASCII_LENGTH("[]boolean"), + '[', ']', 'b', 'o', 'o', 'l', 'e', 'a', 'n' }; binaryurp::Unmarshal m(rtl::Reference< binaryurp::Bridge >(), state, buf); css::uno::TypeDescription t(m.readType()); CPPUNIT_ASSERT( @@ -75,17 +70,14 @@ void Test::testTypeOfBooleanSequence() { void Test::testTypeOfVoidSequence() { binaryurp::ReaderState state; - css::uno::Sequence< sal_Int8 > buf(10); - buf[0] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(20 | 0x80)); // sequence type | cache flag - buf[1] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(binaryurp::cache::ignore >> 8)); - buf[2] = static_cast< sal_Int8 >(static_cast< sal_uInt8 >(binaryurp::cache::ignore & 0xFF)); - buf[3] = RTL_CONSTASCII_LENGTH("[]void"); - buf[4] = '['; - buf[5] = ']'; - buf[6] = 'v'; - buf[7] = 'o'; - buf[8] = 'i'; - buf[9] = 'd'; + css::uno::Sequence buf{ + static_cast(static_cast(20 | 0x80)), + // sequence type | cache flag + static_cast( + static_cast(binaryurp::cache::ignore >> 8)), + static_cast( + static_cast(binaryurp::cache::ignore & 0xFF)), + RTL_CONSTASCII_LENGTH("[]void"), '[', ']', 'v', 'o', 'i', 'd' }; binaryurp::Unmarshal m(rtl::Reference< binaryurp::Bridge >(), state, buf); try { m.readType(); diff --git a/binaryurp/source/bridgefactory.cxx b/binaryurp/source/bridgefactory.cxx index 6b8994dbb152..c9d0a04259eb 100644 --- a/binaryurp/source/bridgefactory.cxx +++ b/binaryurp/source/bridgefactory.cxx @@ -52,8 +52,7 @@ OUString BridgeFactory::static_getImplementationName() { css::uno::Sequence< OUString > BridgeFactory::static_getSupportedServiceNames() { - OUString name("com.sun.star.bridge.BridgeFactory"); - return css::uno::Sequence< OUString >(&name, 1); + return css::uno::Sequence{ "com.sun.star.bridge.BridgeFactory" }; } void BridgeFactory::removeBridge( -- cgit v1.2.3