Hi Tamas,

On Fri, Aug 10, 2012 at 10:19:19AM +0200, [email protected] wrote:
> +    String aString;

I know this is copied from existing code, but if you are at it, can't
you turn this into an OUString? most probably it'll work out of the box,
you just need to rename the class.

(See
http://wiki.documentfoundation.org/Development/String_Classes#Deprecated
)

> -                SetText( aString );
> +                SetText( String() );

Here as well.

> -                if ( sItemCommand == ".uno:FindText" )
> +                if ( sItemCommand == COMMAND_FINDTEXT )

Good catch. :)

Care to resubmit with this change, please?

Thanks,

Miklos
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to