https://bugs.freedesktop.org/show_bug.cgi?id=54938

--- Comment #12 from Marcos Souza <marcos.souza....@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.
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to