Discussion:
Review Request 123474: Use Qt5 syntax for all signals/slots
Jan Grulich
2015-04-23 09:38:57 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123474/
-----------------------------------------------------------

Review request for Network Management, Lukáš Tinkl and Lamarque Souza.


Repository: plasma-nm


Description
-------

This patch transforms old signals/slot syntax to the new one one introduced in Qt5. Not sure if this needs review, because everything compiles fine and thanks to the new syntax all issues would be spotted during compilation, but it's a quite a big patch and would be safer to have someone else to review it.


Diffs
-----

editor/connectioneditor.cpp f2f1418
kded/modemmonitor.cpp c0c4e7a
kded/notification.cpp 283fc00
libs/declarative/availabledevices.cpp 49b468d
libs/declarative/connectionicon.h ae8cceb
libs/declarative/connectionicon.cpp c654217
libs/declarative/enabledconnections.cpp 8a57b50
libs/declarative/networkstatus.cpp 1f830f9
libs/declarative/trafficmonitor.cpp fb14654
libs/editor/settings/bondwidget.cpp cb5028a
libs/editor/settings/bridgewidget.cpp f0e1737
libs/editor/settings/btwidget.cpp ea6f09c
libs/editor/settings/cdmawidget.cpp 1030e78
libs/editor/settings/connectionwidget.cpp 78ee3e5
libs/editor/settings/gsmwidget.cpp 6d1b215
libs/editor/settings/infinibandwidget.cpp 0425c55
libs/editor/settings/ipv4widget.cpp 535a97e
libs/editor/settings/ipv6widget.cpp eb6d6a7
libs/editor/settings/pppoewidget.cpp 3034574
libs/editor/settings/security802-1x.cpp 87f31e4
libs/editor/settings/teamwidget.cpp 8254194
libs/editor/settings/vlanwidget.cpp 8e704ee
libs/editor/settings/wificonnectionwidget.cpp 9a9f763
libs/editor/settings/wifisecurity.cpp ea56b32
libs/editor/settings/wimaxwidget.cpp 8c651b8
libs/editor/settings/wiredconnectionwidget.cpp 054c6ce
libs/editor/settings/wiredsecurity.cpp 5dd59af
libs/editor/widgets/advancedpermissionswidget.cpp e5ea269
libs/editor/widgets/bssidcombobox.cpp 3168577
libs/editor/widgets/hwaddrcombobox.cpp 07537e7
libs/editor/widgets/ipv4routeswidget.cpp e383630
libs/editor/widgets/ipv6routeswidget.cpp b4b5637
libs/editor/widgets/mobileconnectionwizard.cpp 771103f
libs/editor/widgets/ssidcombobox.cpp 628dc5a
libs/handler.cpp 7a6d26c
libs/models/networkmodel.cpp 448ac3c
vpn/l2tp/l2tpauth.cpp 5172e72
vpn/l2tp/l2tpwidget.cpp 2d41ea4
vpn/openconnect/openconnectauth.cpp 1a186e6
vpn/openconnect/openconnectwidget.cpp 8eae8ae
vpn/openswan/openswanauth.cpp 4d66b26
vpn/openswan/openswanwidget.cpp 689c9ac
vpn/openvpn/openvpnadvancedwidget.cpp cd8785d
vpn/openvpn/openvpnauth.cpp e5bef0d
vpn/openvpn/openvpnwidget.cpp 94f79e6
vpn/pptp/pptpauth.cpp e0d1716
vpn/pptp/pptpwidget.cpp 01d18e2
vpn/strongswan/strongswanauth.cpp 5856382
vpn/strongswan/strongswanwidget.cpp 828e535
vpn/vpnc/vpnc.h 748091c
vpn/vpnc/vpnc.cpp f88d206

Diff: https://git.reviewboard.kde.org/r/123474/diff/


Testing
-------

Everything seems to work as before.


Thanks,

Jan Grulich
Lamarque Souza
2015-04-23 12:12:10 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123474/#review79380
-----------------------------------------------------------

Ship it!


Ship It!

- Lamarque Souza
Post by Jan Grulich
-----------------------------------------------------------
https://git.reviewboard.kde.org/r/123474/
-----------------------------------------------------------
(Updated April 23, 2015, 9:38 a.m.)
Review request for Network Management, Lukáš Tinkl and Lamarque Souza.
Repository: plasma-nm
Description
-------
This patch transforms old signals/slot syntax to the new one one introduced in Qt5. Not sure if this needs review, because everything compiles fine and thanks to the new syntax all issues would be spotted during compilation, but it's a quite a big patch and would be safer to have someone else to review it.
Diffs
-----
editor/connectioneditor.cpp f2f1418
kded/modemmonitor.cpp c0c4e7a
kded/notification.cpp 283fc00
libs/declarative/availabledevices.cpp 49b468d
libs/declarative/connectionicon.h ae8cceb
libs/declarative/connectionicon.cpp c654217
libs/declarative/enabledconnections.cpp 8a57b50
libs/declarative/networkstatus.cpp 1f830f9
libs/declarative/trafficmonitor.cpp fb14654
libs/editor/settings/bondwidget.cpp cb5028a
libs/editor/settings/bridgewidget.cpp f0e1737
libs/editor/settings/btwidget.cpp ea6f09c
libs/editor/settings/cdmawidget.cpp 1030e78
libs/editor/settings/connectionwidget.cpp 78ee3e5
libs/editor/settings/gsmwidget.cpp 6d1b215
libs/editor/settings/infinibandwidget.cpp 0425c55
libs/editor/settings/ipv4widget.cpp 535a97e
libs/editor/settings/ipv6widget.cpp eb6d6a7
libs/editor/settings/pppoewidget.cpp 3034574
libs/editor/settings/security802-1x.cpp 87f31e4
libs/editor/settings/teamwidget.cpp 8254194
libs/editor/settings/vlanwidget.cpp 8e704ee
libs/editor/settings/wificonnectionwidget.cpp 9a9f763
libs/editor/settings/wifisecurity.cpp ea56b32
libs/editor/settings/wimaxwidget.cpp 8c651b8
libs/editor/settings/wiredconnectionwidget.cpp 054c6ce
libs/editor/settings/wiredsecurity.cpp 5dd59af
libs/editor/widgets/advancedpermissionswidget.cpp e5ea269
libs/editor/widgets/bssidcombobox.cpp 3168577
libs/editor/widgets/hwaddrcombobox.cpp 07537e7
libs/editor/widgets/ipv4routeswidget.cpp e383630
libs/editor/widgets/ipv6routeswidget.cpp b4b5637
libs/editor/widgets/mobileconnectionwizard.cpp 771103f
libs/editor/widgets/ssidcombobox.cpp 628dc5a
libs/handler.cpp 7a6d26c
libs/models/networkmodel.cpp 448ac3c
vpn/l2tp/l2tpauth.cpp 5172e72
vpn/l2tp/l2tpwidget.cpp 2d41ea4
vpn/openconnect/openconnectauth.cpp 1a186e6
vpn/openconnect/openconnectwidget.cpp 8eae8ae
vpn/openswan/openswanauth.cpp 4d66b26
vpn/openswan/openswanwidget.cpp 689c9ac
vpn/openvpn/openvpnadvancedwidget.cpp cd8785d
vpn/openvpn/openvpnauth.cpp e5bef0d
vpn/openvpn/openvpnwidget.cpp 94f79e6
vpn/pptp/pptpauth.cpp e0d1716
vpn/pptp/pptpwidget.cpp 01d18e2
vpn/strongswan/strongswanauth.cpp 5856382
vpn/strongswan/strongswanwidget.cpp 828e535
vpn/vpnc/vpnc.h 748091c
vpn/vpnc/vpnc.cpp f88d206
Diff: https://git.reviewboard.kde.org/r/123474/diff/
Testing
-------
Everything seems to work as before.
Thanks,
Jan Grulich
Jan Grulich
2015-04-23 12:17:11 UTC
Permalink
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123474/
-----------------------------------------------------------

(Updated April 23, 2015, 12:17 p.m.)


Status
------

This change has been marked as submitted.


Review request for Network Management, Lukáš Tinkl and Lamarque Souza.


Changes
-------

Submitted with commit d418067e5a21b8596df79e251e66abd82d93990f by Jan Grulich to branch master.


Repository: plasma-nm


Description
-------

This patch transforms old signals/slot syntax to the new one one introduced in Qt5. Not sure if this needs review, because everything compiles fine and thanks to the new syntax all issues would be spotted during compilation, but it's a quite a big patch and would be safer to have someone else to review it.


Diffs
-----

editor/connectioneditor.cpp f2f1418
kded/modemmonitor.cpp c0c4e7a
kded/notification.cpp 283fc00
libs/declarative/availabledevices.cpp 49b468d
libs/declarative/connectionicon.h ae8cceb
libs/declarative/connectionicon.cpp c654217
libs/declarative/enabledconnections.cpp 8a57b50
libs/declarative/networkstatus.cpp 1f830f9
libs/declarative/trafficmonitor.cpp fb14654
libs/editor/settings/bondwidget.cpp cb5028a
libs/editor/settings/bridgewidget.cpp f0e1737
libs/editor/settings/btwidget.cpp ea6f09c
libs/editor/settings/cdmawidget.cpp 1030e78
libs/editor/settings/connectionwidget.cpp 78ee3e5
libs/editor/settings/gsmwidget.cpp 6d1b215
libs/editor/settings/infinibandwidget.cpp 0425c55
libs/editor/settings/ipv4widget.cpp 535a97e
libs/editor/settings/ipv6widget.cpp eb6d6a7
libs/editor/settings/pppoewidget.cpp 3034574
libs/editor/settings/security802-1x.cpp 87f31e4
libs/editor/settings/teamwidget.cpp 8254194
libs/editor/settings/vlanwidget.cpp 8e704ee
libs/editor/settings/wificonnectionwidget.cpp 9a9f763
libs/editor/settings/wifisecurity.cpp ea56b32
libs/editor/settings/wimaxwidget.cpp 8c651b8
libs/editor/settings/wiredconnectionwidget.cpp 054c6ce
libs/editor/settings/wiredsecurity.cpp 5dd59af
libs/editor/widgets/advancedpermissionswidget.cpp e5ea269
libs/editor/widgets/bssidcombobox.cpp 3168577
libs/editor/widgets/hwaddrcombobox.cpp 07537e7
libs/editor/widgets/ipv4routeswidget.cpp e383630
libs/editor/widgets/ipv6routeswidget.cpp b4b5637
libs/editor/widgets/mobileconnectionwizard.cpp 771103f
libs/editor/widgets/ssidcombobox.cpp 628dc5a
libs/handler.cpp 7a6d26c
libs/models/networkmodel.cpp 448ac3c
vpn/l2tp/l2tpauth.cpp 5172e72
vpn/l2tp/l2tpwidget.cpp 2d41ea4
vpn/openconnect/openconnectauth.cpp 1a186e6
vpn/openconnect/openconnectwidget.cpp 8eae8ae
vpn/openswan/openswanauth.cpp 4d66b26
vpn/openswan/openswanwidget.cpp 689c9ac
vpn/openvpn/openvpnadvancedwidget.cpp cd8785d
vpn/openvpn/openvpnauth.cpp e5bef0d
vpn/openvpn/openvpnwidget.cpp 94f79e6
vpn/pptp/pptpauth.cpp e0d1716
vpn/pptp/pptpwidget.cpp 01d18e2
vpn/strongswan/strongswanauth.cpp 5856382
vpn/strongswan/strongswanwidget.cpp 828e535
vpn/vpnc/vpnc.h 748091c
vpn/vpnc/vpnc.cpp f88d206

Diff: https://git.reviewboard.kde.org/r/123474/diff/


Testing
-------

Everything seems to work as before.


Thanks,

Jan Grulich

Loading...