[Libreoffice] [PATCH] Error dialog when a component is missing

Kohei Yoshida kyoshida at novell.com
Fri Apr 22 11:18:55 PDT 2011


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.

And yes, the design of this over-complicated error handling is pretty
amazing in a certain way, and .... I'll leave at that.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>



More information about the LibreOffice mailing list