Re: Review Request 113510: Deprecate methods in kimageio

2013-11-03 Thread Alex Merry
On 02/11/13 23:09, David Faure wrote: > On Saturday 02 November 2013 22:36:03 Alex Merry wrote: >> On 02/11/13 19:36, David Faure wrote: >>> If we want to get rid of the .desktop files, why don't we just implement >>> KImageIO on top of QImageReader/Writer, rather than duplicate its code? >> Only t

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
On Saturday 02 November 2013 22:36:03 Alex Merry wrote: > Moving this to the list. > > On 02/11/13 19:36, David Faure wrote: > > On November 2nd, 2013, 6:21 p.m. UTC, *Alex Merry* wrote: > > Related question: would it make sense to try to implement this by > > directly enumerat

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
Moving this to the list. On 02/11/13 19:36, David Faure wrote: > On November 2nd, 2013, 6:21 p.m. UTC, *Alex Merry* wrote: > Related question: would it make sense to try to implement this by > directly enumerating the QImageIO plugins (and reading their JSON files)? > Possibly with

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? > > David Faure wrote: > Sure, that's exactly the idea (I'll do that once this is committed) > > Alex Merry wrote: > I think I'd misse

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42866 --- This review has been submitted with commit 4eb8e017268d0e75a60

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- (Updated Nov. 2, 2013, 7:04 p.m.) Status -- This change has been mark

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? > > David Faure wrote: > Sure, that's exactly the idea (I'll do that once this is committed) I think I'd missed it was still in the "to be

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? Sure, that's exactly the idea (I'll do that once this is committed) - David --- Thi

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! Question: would it not make sense to put this in kde4support? - Alex --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42823 --- Ship it! Ship It! - David Faure On Nov. 1, 2013, 4:51 p.m.,

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-01 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- (Updated Nov. 1, 2013, 10:51 a.m.) Review request for KDE Frameworks. Ch

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-01 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42758 --- Missing #ifndef KDE_NO_DEPRECATED around all the deprecated met

Review Request 113510: Deprecate methods in kimageio

2013-10-30 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- Review request for KDE Frameworks. Repository: kdelibs Description -