[Libreoffice] [PUSHED] Re: [PATCH] Error dialog when a component is missing

Lubos Lunak l.lunak at suse.cz
Tue Apr 26 06:23:35 PDT 2011


On Friday 22 of April 2011, Kohei Yoshida wrote:
> Hi Lubos,
>
> On Fri, 2011-04-22 at 15:36 +0200, Lubos Lunak wrote:
> > Hello,
> >
> >  could somebody review the attached patch for bnc#689116? When e.g.
> > Writer is not installed (which can happen with split rpms) and one tries
> > to open a text document, LO simply quits without saying anything.
> > Guessing from the surrounding code it's correct, but I'd prefer it to be
> > checked as I have no clue about LO UI string handling or error handling
> > (and I bow to the creativitity of whoever has created this complex error
> > handling system).
>
> I'm not an expert of this code either, but your change looks reasonable.
> So go for it. :-)  To be honest I don't think any one of us is
> particularly versed in this part of the code base since it's relatively
> new.

 Pushed.

-- 
 Lubos Lunak
 l.lunak at suse.cz


More information about the LibreOffice mailing list