Re: [linux-minidisc] saving window geometry
On Sat, Jun 05, 2010 at 10:44:09AM +0200, Thomas Arp wrote:
> Hi,
> i added a small feature to save the window geometry and
> column width of the himd browser when closed and restores
> them at next startup.
Looks good to me.
Could you re-format the patch? There are some places where you "patch"
whitespaces like lines 19 and 20 in the patch file:
@@ -154,7 +154,7 @@ QString QHiMDMainWindow::dumppcm(const QHiMDTrack
& track, QString file)
while(himd_nonmp3stream_read_block(&str, &data, &len, NULL,
&status) >= 0)
{
-
+
Also:
void QHiMDMainWindow::save_window_settings()
{
int i = 0;
settings.setValue("geometry", QMainWindow::saveGeometry());
settings.setValue("windowState", QMainWindow::saveState());
for(;i == 6; i++)
Shouldn't we rather read the number of columns in the list view rather
than assuming it's 6? I think using a constant here is more prone to
errors when changing something.
Otherwise I think the patch is fine and I would apply it after you
fixed the suggested changes or come up with some pretty good arguments
;).
Adrian