--MM5RgFPKyuP3gDcV
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Jun 14, 2004 at 12:14:47PM +0200, Lubos Lunak wrote:
> > From  bool saveState( QSessionManager& ) in KMWSessionManaged, around
> > kmainwindow.cpp  line 99:
> >
> >         for (it.toFirst(); it.current(); ++it){
> >             if( it.current()->isVisible()) {
> >                 n++;
> >                 it.current()->savePropertiesInternal(config, n);
> >             }
> >         }
> >
> > Why is the isVisible here?
>=20
>  cvs annotate is your friend :=20
> http://webcvs.kde.org/cgi-bin/cvsweb.cgi/kdelibs/kdeui/kmainwindow.cpp#re=
v1.105
>=20
>  I think it could be reverted and that particular problem fixed directly =
in=20
> Konqueror.

Shoot, should've done so before the beta, can I still revert for 3.3?

It would reopen #59334, but it would fix the regression since 3.1 and I'd
rather re-break Konqueror than continue to break every application depending
on session management.

Rob
--=20
Rob Kaper     | "Whoever wants to know the heart and mind of America had
cap@capsi.com | better learn baseball" -- Jacques Barzun

--MM5RgFPKyuP3gDcV
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQFA7pHutppIl2G1SjcRAuHzAJ9y29x7r2jfXA0DCNn1PSUSf1/ZYgCeMK7I
O4UgDjysbl7seNf1h2r9hh0=
=1g9V
-----END PGP SIGNATURE-----

--MM5RgFPKyuP3gDcV--