Re: Review Request: Potential crash due to notification during file save

2012-05-22 Thread Dag Andersen
> On May 22, 2012, 9:04 a.m., Boudewijn Rempt wrote: > > sheets/CMakeLists.txt, line 332 > > > > > > er, why this? > > Dag Andersen wrote: > You need a .notifyrc if you actually want KNotification to issue > a

Re: Review Request: Potential crash due to notification during file save

2012-05-22 Thread Boudewijn Rempt
> On May 22, 2012, 9:04 a.m., Boudewijn Rempt wrote: > > sheets/CMakeLists.txt, line 332 > > > > > > er, why this? > > Dag Andersen wrote: > You need a .notifyrc if you actually want KNotification to issue > a

Re: Review Request: Potential crash due to notification during file save

2012-05-22 Thread Dag Andersen
> On May 22, 2012, 9:04 a.m., Boudewijn Rempt wrote: > > sheets/CMakeLists.txt, line 332 > > > > > > er, why this? You need a .notifyrc if you actually want KNotification to issue a notification, sending a dbus me

Re: Review Request: Potential crash due to notification during file save

2012-05-22 Thread Boudewijn Rempt
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105010/#review14031 --- Ship it! Good research! Just one question: sheets/CMakeLists

Review Request: Potential crash due to notification during file save

2012-05-22 Thread Dag Andersen
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105010/ --- Review request for Calligra. Description --- AFAICS any calligra app