----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101788/#review4212 -----------------------------------------------------------
vpnplugins/openconnect/README <http://git.reviewboard.kde.org/r/101788/#comment3458> You can remove the parentheses. vpnplugins/openconnect/openconnectauth.cpp <http://git.reviewboard.kde.org/r/101788/#comment3460> You can change this to: "Check failed for certificate from VPN server \"%1\".\n" "Reason: %2\nAccept it anyway?" vpnplugins/openconnect/openconnectauth.cpp <http://git.reviewboard.kde.org/r/101788/#comment3459> "from the form" vpnplugins/openconnect/openconnectauth.cpp <http://git.reviewboard.kde.org/r/101788/#comment3461> "unsuccessful" vpnplugins/openconnect/openconnectauth.cpp <http://git.reviewboard.kde.org/r/101788/#comment3464> Let's hope this recursive delete does not do the opposite hehe. vpnplugins/openconnect/openconnectauth.cpp <http://git.reviewboard.kde.org/r/101788/#comment3462> I think you should use deleteLater here too. All children of item will be deleted by this delete call. If item->widget() is a child of item (I guess it is) then the deleteLater call above will not be executed because item->widget will be already deleted here. - Lamarque Vieira On June 27, 2011, 7:55 p.m., Ilia Kats wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101788/ > ----------------------------------------------------------- > > (Updated June 27, 2011, 7:55 p.m.) > > > Review request for Network Management. > > > Summary > ------- > > Add an OpenConnect VPN plug-in to KDE NM. First time using threads, so I > don't know if that's how it's supposed to be done, but it seems to be working. > Also, I can't figure out how to make the loginForm QGroupBox have a minimum > height, but get bigger when widgets get added. If the minimum height is 0, > then both the box and the dialog get resized, however this causes visual > "interference" as the serverLogBox jumps up and down as the upper loginForm > box gets resized. Setting the minimum height of the loginForm box to 100 > causes it to stay at 100 and squeeze the added widgets to fit the size, > instead of getting bigger. Any ideas? > > This requires OpenConnect >= 3.03 to build. 3.03 is not yet released, but the > important thing is commit 423eee0b51a204562d6f2ec67893133ebcf200d6 from > OpenConnect git (http://git.infradead.org/users/dwmw2/openconnect.git/ ) > > > This addresses bug 226028. > http://bugs.kde.org/show_bug.cgi?id=226028 > > > Diffs > ----- > > libs/internals/settings/vpnsecrets.cpp 0d9e3f9 > libs/ui/connectionsecretsjob.cpp d791bb3 > libs/ui/vpnuiplugin.h 444ab2a > libs/ui/vpnuiplugin.cpp d058a52 > vpnplugins/CMakeLists.txt 4706a61 > vpnplugins/openconnect/CMakeLists.txt PRE-CREATION > vpnplugins/openconnect/FindOpenConnect.cmake PRE-CREATION > vpnplugins/openconnect/README PRE-CREATION > vpnplugins/openconnect/networkmanagement_openconnectui.desktop PRE-CREATION > vpnplugins/openconnect/nm-openconnect-service.h PRE-CREATION > vpnplugins/openconnect/openconnectauth.h PRE-CREATION > vpnplugins/openconnect/openconnectauth.cpp PRE-CREATION > vpnplugins/openconnect/openconnectauth.ui PRE-CREATION > vpnplugins/openconnect/openconnectauthworkerthread.h PRE-CREATION > vpnplugins/openconnect/openconnectauthworkerthread.cpp PRE-CREATION > vpnplugins/openconnect/openconnectprop.ui PRE-CREATION > vpnplugins/openconnect/openconnectui.h PRE-CREATION > vpnplugins/openconnect/openconnectui.cpp PRE-CREATION > vpnplugins/openconnect/openconnectwidget.h PRE-CREATION > vpnplugins/openconnect/openconnectwidget.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/101788/diff > > > Testing > ------- > > Yes, see the bugzilla ticket. > > > Screenshots > ----------- > > > http://git.reviewboard.kde.org/r/101788/s/191/ > > > Thanks, > > Ilia > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
