[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: Re: [PATCH] KConfig XT and koshell
From: Sven_Lüppken <sven () kde ! org>
Date: 2005-02-18 22:11:31
Message-ID: 42166813.1010707 () kde ! org
[Download RAW message or body]
Hi Aaron!
> +void KoShellWindow::saveSettings()
> +{
> + KoShellSettings::setSidebarWidth( m_pKoolBar->width() );
> KoShellSettings::writeConfig();
> +}
>
> as an aside, why bother keeping m_width around at all? just access
> KoShellSettings::sidebarWidth() everywhere. save yourself have to sync the
> two values and save a whopping whole sizeof(bool) of memory ;P
Thanks for your help, patch is committed now :)
Regards,
Sven Lüppken
sven@kde.org
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic