Re: Review Request 126770: Fix most Clazy warnings in KXmlGui.

2016-10-20 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit d4d4796620eb850ee6d305d9bd3478e6b7cb216d by Albert Astals Cid on behalf of Andrey Cygankov to branch master. Repository: kxmlgui Description --- Fix all Clazy warnings

Re: Review Request 126765: Fix some Clazy warnings in KTextEditor.

2016-02-28 Thread Andrey Cygankov
/normalvimode.cpp c0eaf40 src/vimode/modes/visualvimode.cpp c58277b src/vimode/searcher.cpp 133ff9b Diff: https://git.reviewboard.kde.org/r/126765/diff/ Testing --- Built without errors. All tests passed. Thanks, Andrey Cygankov ___ Kde

Re: Review Request 126769: Fix all Clazy warnings in KDED.

2016-02-28 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 706bf6f39319f6c2b79ad82c689d7506747b98c7 by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kded Description --- Fix all Clazy warnings in KDED

Re: Review Request 126772: Fix some Clazy warnings in KService

2016-02-28 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 1939360c50cba1241b01108fc32e73ef88c1d83d by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kservice Description --- Fix some Clazy warnings

Review Request 126765: Fix some Clazy warnings in KTextEditor.

2016-01-16 Thread Andrey Cygankov
. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 126769: Fix all Clazy warnings in KDED.

2016-01-16 Thread Andrey Cygankov
: kded Description --- Fix all Clazy warnings in KDED. Diffs - src/kded.cpp 0eb6380 Diff: https://git.reviewboard.kde.org/r/126769/diff/ Testing --- Built without errors. No tests available. Thanks, Andrey Cygankov ___ Kde

Review Request 126770: Fix most Clazy warnings in KXmlGui.

2016-01-16 Thread Andrey Cygankov
ktoolbar.cpp 8fcb9cb src/kxmlguifactory_p.cpp 5e1a0d6 tests/kwindowtest.cpp 0555a56 Diff: https://git.reviewboard.kde.org/r/126770/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing

Review Request 126772: Fix some Clazy warnings in KService

2016-01-16 Thread Andrey Cygankov
patch: Failed kmimeassociationstest With patch: Failed kmimeassociationstest Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
: > > `/home/kde-devel/frameworks/kitemviews/src/kwidgetitemdelegatepool.cpp:48:1: > > warning: KWidgetItemDelegateEventListener is missing a Q_OBJECT macro > > [-Wclazy-missing-qobject]` > > Andrey Cygankov wrote: > Strangely, I had not noticed the warning in the log .

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-14 Thread Andrey Cygankov
/selectionpmwidget.cpp f5b353a Diff: https://git.reviewboard.kde.org/r/126735/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
ing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
. I get to the PC - see the log again. - Andrey --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126737/#review91067 -------

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
: > > `/home/kde-devel/frameworks/kitemviews/src/kwidgetitemdelegatepool.cpp:48:1: > > warning: KWidgetItemDelegateEventListener is missing a Q_OBJECT macro > > [-Wclazy-missing-qobject]` > > Andrey Cygankov wrote: > Strangely, I had not noticed the warning in the

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
: > > `/home/kde-devel/frameworks/kitemviews/src/kwidgetitemdelegatepool.cpp:48:1: > > warning: KWidgetItemDelegateEventListener is missing a Q_OBJECT macro > > [-Wclazy-missing-qobject]` > > Andrey Cygankov wrote: > Strangely, I had not noticed the warning in the log .

Re: Review Request 126670: Fix some Clazy warnings in KPeople framework

2016-01-14 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 7f896240d3be09e33d76a8ae8be953c498c662e7 by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kpeople Description --- Fix some Clazy warnings

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
n automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126737/#review91076 ------- On Янв. 14, 2016, 10:40 п.п., Andrey Cygankov wrote: > > --- > This is

Re: Review Request 126736: Fix all Clazy warnings in KFileMetadata.

2016-01-14 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 27427ffebdd82141dc071423b6ef37c8a69119b0 by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kfilemetadata Description --- Fixed all Clazy warnings

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks, Aleix Pol Gonzalez and Sergio Martins. Changes --- Submitted with commit 69b721e73b421c1950ca32f1e774471060f2d4fe by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kitemviews Description --- Fix Clazy

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
iff: https://git.reviewboard.kde.org/r/126737/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-14 Thread Andrey Cygankov
> On Янв. 14, 2016, 12:07 д.п., Aleix Pol Gonzalez wrote: > > These look good. You wrote "most". Which warnings couldn't you fix? > > Andrey Cygankov wrote: > log file sent > > Aleix Pol Gonzalez wrote: > > `/home/andrey/dev/projects/k

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-14 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 1e487951342a79eff5c680d29c7e9ed852a47dc4 by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kitemmodels Description --- Fixed most Clazy warnings

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
: > > `/home/kde-devel/frameworks/kitemviews/src/kwidgetitemdelegatepool.cpp:48:1: > > warning: KWidgetItemDelegateEventListener is missing a Q_OBJECT macro > > [-Wclazy-missing-qobject]` > > Andrey Cygankov wrote: > Strangely, I had not noticed the warning in the log .

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-14 Thread Andrey Cygankov
/selectionpmwidget.cpp f5b353a Diff: https://git.reviewboard.kde.org/r/126735/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo

Re: Review Request 126737: Fix most Clazy warnings in KItemViews.

2016-01-14 Thread Andrey Cygankov
: > > `/home/kde-devel/frameworks/kitemviews/src/kwidgetitemdelegatepool.cpp:48:1: > > warning: KWidgetItemDelegateEventListener is missing a Q_OBJECT macro > > [-Wclazy-missing-qobject]` > > Andrey Cygankov wrote: > Strangely, I had not noticed the warning in the log .

Re: Review Request 126734: Fix Clazy warnings in KNotifications.

2016-01-13 Thread Andrey Cygankov
/knotificationrestrictions.cpp c979a78 src/notifybypopup.cpp e3090ba src/notifybypopupgrowl.cpp 504ad90 Diff: https://git.reviewboard.kde.org/r/126734/diff/ Testing --- Built without errors. No tests available. Thanks, Andrey Cygankov ___ Kde

Re: Review Request 126734: Fix Clazy warnings in KNotifications.

2016-01-13 Thread Andrey Cygankov
ritemdbus_.cpp ? - Andrey --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126734/#review91034 --- On Янв. 13, 2016, 11:30

Re: Review Request 126734: Fix Clazy warnings in KNotifications.

2016-01-13 Thread Andrey Cygankov
/kstatusnotifieritemdbus_p.cpp b6933b3 src/notifybypopup.cpp e3090ba src/notifybypopupgrowl.cpp 504ad90 Diff: https://git.reviewboard.kde.org/r/126734/diff/ Testing --- Built without errors. No tests available. Thanks, Andrey Cygankov ___ Kde-frameworks-devel

Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-13 Thread Andrey Cygankov
errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126734: Fix Clazy warnings in KNotifications.

2016-01-13 Thread Andrey Cygankov
/knotificationrestrictions.cpp c979a78 src/kstatusnotifieritem.cpp f9bf460 src/notifybypopup.cpp e3090ba src/notifybypopupgrowl.cpp 504ad90 Diff: https://git.reviewboard.kde.org/r/126734/diff/ Testing --- Built without errors. No tests available. Thanks, Andrey Cygankov

Review Request 126736: Fix all Clazy warnings in KFileMetadata.

2016-01-13 Thread Andrey Cygankov
78c4b37 src/extractors/office2007extractor.cpp 548763d src/extractors/officeextractor.cpp efad5d4 tests/dump.cpp 6adb3a9 Diff: https://git.reviewboard.kde.org/r/126736/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-13 Thread Andrey Cygankov
/126735/diff/ Testing --- Build without errors. Tests passed. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126735: Fix most Clazy warnings in KItemModels.

2016-01-13 Thread Andrey Cygankov
reply, visit: https://git.reviewboard.kde.org/r/126735/#review91040 --- On Янв. 14, 2016, 12:05 д.п., Andrey Cygankov wrote: > > --- > This is an automatically generated e-ma

Re: Review Request 126670: Fix some Clazy warnings in KPeople framework

2016-01-08 Thread Andrey Cygankov
> On Янв. 8, 2016, 1:20 д.п., Aleix Pol Gonzalez wrote: > > I did it myself because you're a GCI student, but besides compilation, you > > should have checked if tests still pass and specified so in the testing > > field. > > > > The change looks, good. T

Re: Review Request 126670: Fix some Clazy warnings in KPeople framework

2016-01-07 Thread Andrey Cygankov
src/widgets/persondetailsview.cpp 7e4e24f Diff: https://git.reviewboard.kde.org/r/126670/diff/ Testing --- Compiling without errors. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Re: Review Request 126670: Fix some Clazy warnings in KPeople framework

2016-01-07 Thread Andrey Cygankov
cpp 5c3d3e1 > src/personpluginmanager.cpp f5396d9 > src/widgets/actions.cpp ffaf16f > src/widgets/mergedelegate.cpp 01c988d > src/widgets/persondetailsview.cpp 7e4e24f > > Diff: https://git.reviewboard.kde.org/r/126670/diff/ > > > Testing > --- > &g

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-28 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit d9e679e4f60d2c21c2d256c1a343a38b2dfa9a6f by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kcoreaddons Description --- Fix some Clazy warnings

Re: Review Request 126531: Fix some Clazy warnings in kemoticons.

2015-12-27 Thread Andrey Cygankov
--- Compiling without errors. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126531: Fix some Clazy warnings in kemoticons.

2015-12-27 Thread Andrey Cygankov
s an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126531/#review90163 ------- On Дек. 27, 2015, 5:50 п.п., Andrey Cygankov wrote: > > --- >

Re: Review Request 126531: Fix some Clazy warnings in kemoticons.

2015-12-27 Thread Andrey Cygankov
without errors. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126531: Fix some Clazy warnings in kemoticons.

2015-12-27 Thread Andrey Cygankov
--- Compiling without errors. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126531: Fix some Clazy warnings in kemoticons.

2015-12-27 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit 6e58d0592b8dc70384223ea2a598f7cf776685be by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kemoticons Description --- Fix some Clazy warnings

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-24 Thread Andrey Cygankov
26392/diff/ Testing --- Compiling without errors Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-24 Thread Andrey Cygankov
ed e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126392/#review90049 --- On Дек. 24, 2015, 11:58 д.п., Andrey Cygankov wrote: > > --- > This is an automatically generate

Re: Review Request 126440: Fix some Clazy warnings in kinit.

2015-12-23 Thread Andrey Cygankov
marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes --- Submitted with commit c0c8cd8bca239722d67e1d2e6d4c59ca50671d52 by Aleix Pol on behalf of Andrey Cygankov to branch master. Repository: kinit Description --- Fix some Clazy warnings in kinit

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-23 Thread Andrey Cygankov
n automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126392/#review89994 --- On Дек. 18, 2015, 7:32 п.п., Andrey Cygankov wrote: > > --- > This is an a

Review Request 126440: Fix some Clazy warnings in kinit.

2015-12-20 Thread Andrey Cygankov
/klauncher/klauncher_main.cpp 432e19a src/wrapper.cpp 95b7ec2 Diff: https://git.reviewboard.kde.org/r/126440/diff/ Testing --- Compiling without errors. Automated tests pass. Thanks, Andrey Cygankov ___ Kde-frameworks-devel mailing list Kde

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-20 Thread Andrey Cygankov
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126392/#review89786 --- Hey, when my patch will be reviewed? - Andrey Cygankov

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-18 Thread Andrey Cygankov
dia.org/wiki/De_Morgan's_laws Yes , I realized the error , correct it soon. - Andrey --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126392/#review89700 -----

Re: Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-18 Thread Andrey Cygankov
src/lib/text/kmacroexpander.cpp 0d05ffd src/lib/text/kstringhandler.cpp 826ddcb src/lib/text/ktexttohtml.cpp d58f5c0 src/lib/util/kshell.cpp 11892ce Diff: https://git.reviewboard.kde.org/r/126392/diff/ Testing --- Compiling without errors Thanks, Andrey Cygankov

Review Request 126392: Fix some Clazy warnings in kcoreaddons

2015-12-17 Thread Andrey Cygankov
/lib/text/kstringhandler.cpp 826ddcb src/lib/text/ktexttohtml.cpp d58f5c0 src/lib/util/kshell.cpp 11892ce Diff: https://git.reviewboard.kde.org/r/126392/diff/ Testing --- Compiling without errors Thanks, Andrey Cygankov ___ Kde