Re: Review Request 123735: version of QmlObject with a static engine

2015-06-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated June 10, 2015, 4:41 p.m.) Status -- This change has been

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-23 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80755 --- Ship it! ...in 10 days Great work on all this. - David

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 21, 2015, 10:52 a.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80695 --- Ship it! Awesome. I love the test!

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 21, 2015, 4:02 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-20 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80669 --- src/quickaddons/quickviewsharedengine.h (line 41)

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-20 Thread Marco Martin
On May 20, 2015, 5:24 p.m., Vishesh Handa wrote: src/quickaddons/quickviewsharedengine.h, line 41 https://git.reviewboard.kde.org/r/123735/diff/8/?file=370104#file370104line41 It would be awesome if we could have some tests for this class. Maybe we can just try and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-19 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80630 --- src/kdeclarative/qmlobjectsharedengine.cpp (line 41)

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-19 Thread Vishesh Handa
On May 19, 2015, 3:04 p.m., Vishesh Handa wrote: src/kdeclarative/qmlobjectsharedengine.cpp, line 60 https://git.reviewboard.kde.org/r/123735/diff/8/?file=370102#file370102line60 I'm probably missing some parts of the picture. Could you please explain why this needs to be static?

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-19 Thread Marco Martin
On May 19, 2015, 3:04 p.m., Vishesh Handa wrote: src/kdeclarative/qmlobjectsharedengine.cpp, line 60 https://git.reviewboard.kde.org/r/123735/diff/8/?file=370102#file370102line60 I'm probably missing some parts of the picture. Could you please explain why this needs to be static?

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-19 Thread Marco Martin
On May 19, 2015, 3:04 p.m., Vishesh Handa wrote: src/kdeclarative/qmlobjectsharedengine.cpp, line 60 https://git.reviewboard.kde.org/r/123735/diff/8/?file=370102#file370102line60 I'm probably missing some parts of the picture. Could you please explain why this needs to be static?

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-18 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 18, 2015, 7:09 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-18 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 18, 2015, 7:02 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80600 --- src/quickaddons/quickviewsharedengine.cpp (lines 35 - 39)

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-18 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 18, 2015, 8 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-13 Thread Marco Martin
On May 12, 2015, 5:45 p.m., Mark Gaiser wrote: src/kdeclarative/qmlobjectsharedengine.h, line 57 https://git.reviewboard.kde.org/r/123735/diff/5/?file=368396#file368396line57 std::unique_ptr... ... then you can also forget about the delete in the destructor. buh, fine On

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-13 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 13, 2015, 5:37 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 12, 2015, 4:10 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Marco Martin
On May 12, 2015, 3:52 p.m., David Edmundson wrote: src/kdeclarative/qmlobjectsharedengine.cpp, line 62 https://git.reviewboard.kde.org/r/123735/diff/2/?file=368373#file368373line62 this needs to be initialised. line 65 QQmlEngine *QmlObjectSharedEnginePrivate::s_engine = 0; On

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 12, 2015, 4:05 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80252 --- src/kdeclarative/qmlobjectsharedengine.h (line 57)

Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- Review request for KDE Frameworks and Plasma. Repository: kdeclarative

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80241 --- Can you make the diff against the branch root instead of

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 12, 2015, 3:35 p.m.) Review request for KDE Frameworks and

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80244 --- src/kdeclarative/qmlobject.cpp (line 176)

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80249 --- src/kdeclarative/qmlobject.h (line 81)