Re: Review Request 127209: Fix some issues found by Coverity

2016-03-21 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/ --- (Updated March 22, 2016, 2:50 a.m.) Status -- This change has been m

Re: Review Request 127209: Fix some issues found by Coverity

2016-03-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/ --- (Updated March 3, 2016, 12:16 p.m.) Review request for KDE Frameworks.

Re: Review Request 127209: Fix some issues found by Coverity

2016-03-01 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/#review92968 --- src/lib/io/kdirwatch.cpp (line 181)

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-29 Thread Aleix Pol Gonzalez
> On Feb. 29, 2016, 4:30 p.m., Kai Uwe Broulik wrote: > > src/lib/io/kprocess_p.h, line 33 > > > > > > Why not have a constructor that takes the q ptr? > > > > The only code that uses it does: > > >

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-29 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/#review92912 --- src/lib/io/kprocess_p.h (line 33)

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-29 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/ --- (Updated Feb. 29, 2016, 2:29 p.m.) Review request for KDE Frameworks. R

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-28 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/#review92881 --- You lost the changes to the other files, in the diff. (missin

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/ --- (Updated Feb. 29, 2016, 1:13 a.m.) Review request for KDE Frameworks. C

Re: Review Request 127209: Fix some issues found by Coverity

2016-02-28 Thread Lamarque Souza
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/#review92862 --- src/lib/kaboutdata.cpp (line 993)

Review Request 127209: Fix some issues found by Coverity

2016-02-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127209/ --- Review request for KDE Frameworks. Repository: kcoreaddons Description