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