https://bugs.documentfoundation.org/show_bug.cgi?id=97087

--- Comment #24 from Michael Meeks <michael.me...@collabora.com> ---
Hi Tor,

> Another thing: We should change the name field to be an OUString. Using a 
> const
> char * is just fallback to C thinking and micro-optimisation

I disagree. The joy of a char * is that we can pass a NULL pointer in the
non-debug case - without needing any macro horrors; and avoid calling a
pointless empty OUString constructor =)

    OUString()
    {
        pData = NULL;
        rtl_uString_new( &pData );
    }

Thanks,

Michael.

PS. - I see no problem with merging the existing patches - and then updating
them with a new easy hack to Tor's scheme =)

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to