[MPlayer-cvslog] r20137 - trunk/Gui/win32/dialogs.c

[Dominik 'Rathann' Mierzejewski]

On Monday, 09 October 2006 at 20:21, vayne wrote:
> Author: vayne
> Date: Mon Oct  9 20:21:50 2006
> New Revision: 20137
> 
> Modified:
>    trunk/Gui/win32/dialogs.c
> 
> Log:
> fixed a crash bug as a result of the last change, as well as file skipping (hopefully)
> 
> Modified: trunk/Gui/win32/dialogs.c
> ==============================================================================
> --- trunk/Gui/win32/dialogs.c	(original)
> +++ trunk/Gui/win32/dialogs.c	Mon Oct  9 20:21:50 2006
> @@ -531,9 +531,9 @@
>                  case ID_TRACKLIST:
>                      if(HIWORD(wParam) == LBN_DBLCLK)
>                      {
> -                        if(guiIntfStruct.Playing && selected)
> -                            pl->current = selected - 2;
> -                        else if(selected) pl->current = selected - 1;
> +                        if(selected) pl->current = selected - 1;
> +                            mplSetFileName(NULL, pl->tracks[pl->current]->filename, STREAMTYPE_STREAM);

This indentation is misleading. mplSetFileName call is independent of the
condition above.

> +                        mplGotoTheNext = 0;
>                          gui->startplay(gui);
>                      }
>                      return 0;
> @@ -552,6 +552,7 @@
>                  case ID_PLAY:
>                  {
>                      if(selected) pl->current = selected - 1;
> +                        mplSetFileName(NULL, pl->tracks[pl->current]->filename, STREAMTYPE_STREAM);

Same here.

Regards,
R.

-- 
MPlayer developer and RPMs maintainer: http://rpm.greysector.net/mplayer/
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
	-- from "Collected Sayings of Muad'Dib" by the Princess Irulan