[Bug 54938] Adapt supportsService implementations to cppu::supportsService
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Thu Oct 24 19:09:43 CEST 2013
https://bugs.freedesktop.org/show_bug.cgi?id=54938
--- Comment #12 from Marcos Souza <marcos.souza.org at gmail.com> ---
I saw a lot of places like this:
sal_Bool SAL_CALL ScTabViewObj::supportsService( const OUString& rServiceName )
throw(uno::RuntimeException)
{
return rServiceName.equalsAscii( SCTABVIEWOBJ_SERVICE ) ||
rServiceName.equalsAscii( SCVIEWSETTINGS_SERVICE );
}
So, can we just exchange this to cppu::supportsService? Or we need to some
another pieces of code?
OBS: I believe all places that test the supportsService like this have this
same function:
uno::Sequence<OUString> SAL_CALL ScTabViewObj::getSupportedServiceNames()
throw(uno::RuntimeException)
{
uno::Sequence<OUString> aRet(2);
OUString* pArray = aRet.getArray();
pArray[0] = OUString( SCTABVIEWOBJ_SERVICE );
pArray[1] = OUString( SCVIEWSETTINGS_SERVICE );
return aRet;
}
with the same service name added into uno::Sequence.
With this change we can have a more standard code, at least talking about the
way we get uno service names :)
--
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20131024/7bdd1181/attachment.html>
More information about the LibreOffice
mailing list