[haiku-commits] Re: r36391 - haiku/trunk/src/apps/mediaplayer

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 21 Apr 2010 17:18:00 +0200

On 2010-04-21 at 14:23:08 [+0200], Stephan Assmus <superstippi@xxxxxx> wrote:
> 
> -------- Original-Nachricht --------
> > Datum: Wed, 21 Apr 2010 10:58:18 +0200 (CEST)
> > Von: axeld@xxxxxxxxxxxxxxxx
> > An: haiku-commits@xxxxxxxxxxxxx
> > Betreff: [haiku-commits] r36391 - haiku/trunk/src/apps/mediaplayer
> 
> > Author: axeld
> > Date: 2010-04-21 10:58:18 +0200 (Wed, 21 Apr 2010)
> > New Revision: 36391
> > Changeset: http://dev.haiku-os.org/changeset/36391/haiku
> > 
> > Modified:
> >    haiku/trunk/src/apps/mediaplayer/MainWin.cpp
> > Log:
> > * Cleanup.
> 
> >              if (!hadNext) {
> > -                if (fHasVideo) {
> > -                    if (fCloseWhenDonePlayingMovie)
> > -                        PostMessage(B_QUIT_REQUESTED);
> > -                } else {
> > -                    if (fCloseWhenDonePlayingSound)
> > -                        PostMessage(B_QUIT_REQUESTED);
> > -                }
> > +                if ((fHasVideo && fCloseWhenDonePlayingMovie)
> > +                    || (fHasAudio && fCloseWhenDonePlayingSound))
> > +                    PostMessage(B_QUIT_REQUESTED);
> >              }
> 
> I have not yet looked closely at the rest of the patch,
[...]

When you have, please don't forget to add yourself as reviewer:

  http://dev.haiku-os.org/wiki/R1/Alpha2/MergeTracking

CU, Ingo

Other related posts: