Review Request: various fixes for adhoc support

Lamarque Vieira Souza lamarque at gmail.com
Tue Apr 19 23:42:06 CEST 2011



> On April 19, 2011, 9:18 p.m., Lamarque Vieira Souza wrote:
> > libs/internals/wirelessinterfaceconnectionhelpers.cpp, line 121
> > <http://git.reviewboard.kde.org/r/101158/diff/1/?file=14772#file14772line121>
> >
> >     qobject_cast
> 
> Ilia Kats wrote:
>     Knm::Setting and its child classes do not use the Q_OBJECT macro, so no qobject_cast here

Nevermind, this class is not a Q_OBJECT class, qobject_cast does not work here.


- Lamarque Vieira


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101158/#review2754
-----------------------------------------------------------


On April 19, 2011, 9:40 p.m., Ilia Kats wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101158/
> -----------------------------------------------------------
> 
> (Updated April 19, 2011, 9:40 p.m.)
> 
> 
> Review request for Network Management.
> 
> 
> Summary
> -------
> 
> Various fixes for adhoc support. These include:
> - don't show the network we just created twice (once as connection and once as network)
> - show correct security icon for adhoc networks
> - only show strenghtmeter if strength >=0
> - unconnected adhoc networks are now only visible after a click on "Show More...", are always visible if connected and become invisible again after disconnecting
> 
> 
> Diffs
> -----
> 
>   applet/activatableitem.cpp 860b421 
>   applet/activatablelistwidget.h 2e16db3 
>   applet/activatablelistwidget.cpp 63fb8c4 
>   applet/wirelessnetworkitem.cpp 86a7246 
>   libs/internals/settings/802-11-wireless-securitypersistence.cpp ef93f2c 
>   libs/internals/wirelessinterfaceconnectionhelpers.cpp 55d8527 
>   libs/service/wirelessnetworkinterfaceactivatableprovider.cpp 482727d 
>   libs/ui/802_11_wirelesswidget.cpp af67199 
> 
> Diff: http://git.reviewboard.kde.org/r/101158/diff
> 
> 
> Testing
> -------
> 
> Yes
> 
> 
> Thanks,
> 
> Ilia
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-networkmanager/attachments/20110419/07cbf4d8/attachment.htm 


More information about the kde-networkmanager mailing list