[prev in list] [next in list] [prev in thread] [next in thread]
List: helix-player-dev
Subject: [Player-dev] Re: [Midplayer-private-dev] CR: [#8338] Fixing can't
From: Kinson Liu <kliu () real ! com>
Date: 2008-09-25 0:00:56
Message-ID: 48DAD4B8.9070502 () real ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Thanks, checked into 310atlas and HEAD.
Kinson
Rusty Lynch wrote:
> Looks good to me
>
> On Tue, 2008-09-23 at 18:11 -0700, Kinson Liu wrote:
>
>> Modified by: kliu@real.com
>> Date: 9/23/2008
>> Project: RealPlayer for MID
>> Bug Number: 8338
>> Bug URL: https://bugs.helixcommunity.org/show_bug.cgi?id=8338
>>
>> Synopsis: Fixing can't edit or delete an Online Music file if it was
>> opened last time
>>
>> Overview: When the player deletes, it checks against the pl.selected
>> variable to see if the audio is currently playing back. However, what
>> pl.selected actually means is the audio of that playlist lastly played
>> back. When an audio A in Online Music is chosen to play back,
>> pl.selected of Online Music is that audio A. Then the playlist is
>> switched to All Songs and an audio B is chosen to play back;
>> pl.selected of All Songs is B. Finally, the playlist is switched back
>> to Online Music and audio A once again gets selected and is being
>> deleted, the reported error occurs because pl.selected of Online Music
>> is still holding A. Fix for this is to have the player check against
>> self.cur_playlist.selected, which marks the realtime playback audio of
>> all playlists, instead of pl.selected.
>>
>> Files Added:
>> None
>>
>> Files Modified:
>> player/mid/src/music_view.py
>>
>> Image Size and Heap Use impact (Client -Only):
>> None
>>
>> Platforms and Profiles Affected:
>> Platform: linux-2.2-libc6-gcc32-i586
>> Profile: Helix-client-moblin
>>
>> Distribution Libraries Affected:
>> None
>>
>> Distribution library impact and planned action:
>> None
>>
>> Platforms and Profiles Build Verified:
>> Platform: linux-2.2-libc6-gcc32-i586
>> Profile: Helix-client-moblin
>>
>> Platforms and Profiles Functionality verified:
>> Platform: linux-2.2-libc6-gcc32-i586
>> Profile: Helix-client-moblin
>>
>> Branch: hxclient_3_4_1_atlas_restricted
>>
>> Copyright assignment: I am a RealNetworks employee or contractor
>> --
>>
>> ______________________________________________________________________
>> Kinson Liu | Software Development Engineer | RealNetworks, Inc. |
>> kliu@real.com | 206.892.6177
>>
>
>
>
--
------------------------------------------------------------------------
*Kinson Liu* | Software Development Engineer | RealNetworks, Inc. |
kliu@real.com | 206.892.6177
[Attachment #5 (text/html)]
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Thanks, checked into 310atlas and HEAD.<br>
<br>
Kinson<br>
<br>
Rusty Lynch wrote:
<blockquote cite="mid:1222290414.32283.2.camel@rusty-laptop" type="cite">
<pre wrap="">Looks good to me
On Tue, 2008-09-23 at 18:11 -0700, Kinson Liu wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Modified by: <a class="moz-txt-link-abbreviated" \
href="mailto:kliu@real.com">kliu@real.com</a>
Date: 9/23/2008
Project: RealPlayer for MID
Bug Number: 8338
Bug URL: <a class="moz-txt-link-freetext" \
href="https://bugs.helixcommunity.org/show_bug.cgi?id=8338">https://bugs.helixcommunity.org/show_bug.cgi?id=8338</a>
Synopsis: Fixing can't edit or delete an Online Music file if it was
opened last time
Overview: When the player deletes, it checks against the pl.selected
variable to see if the audio is currently playing back. However, what
pl.selected actually means is the audio of that playlist lastly played
back. When an audio A in Online Music is chosen to play back,
pl.selected of Online Music is that audio A. Then the playlist is
switched to All Songs and an audio B is chosen to play back;
pl.selected of All Songs is B. Finally, the playlist is switched back
to Online Music and audio A once again gets selected and is being
deleted, the reported error occurs because pl.selected of Online Music
is still holding A. Fix for this is to have the player check against
self.cur_playlist.selected, which marks the realtime playback audio of
all playlists, instead of pl.selected.
Files Added:
None
Files Modified:
player/mid/src/music_view.py
Image Size and Heap Use impact (Client -Only):
None
Platforms and Profiles Affected:
Platform: linux-2.2-libc6-gcc32-i586
Profile: Helix-client-moblin
Distribution Libraries Affected:
None
Distribution library impact and planned action:
None
Platforms and Profiles Build Verified:
Platform: linux-2.2-libc6-gcc32-i586
Profile: Helix-client-moblin
Platforms and Profiles Functionality verified:
Platform: linux-2.2-libc6-gcc32-i586
Profile: Helix-client-moblin
Branch: hxclient_3_4_1_atlas_restricted
Copyright assignment: I am a RealNetworks employee or contractor
--
______________________________________________________________________
Kinson Liu | Software Development Engineer | RealNetworks, Inc. |
<a class="moz-txt-link-abbreviated" href="mailto:kliu@real.com">kliu@real.com</a> | \
206.892.6177 </pre>
</blockquote>
<pre wrap=""><!---->
</pre>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<hr><b>Kinson Liu</b> | Software Development Engineer | RealNetworks,
Inc. | <a class="moz-txt-link-abbreviated" \
href="mailto:kliu@real.com">kliu@real.com</a> | 206.892.6177 </div>
</body>
</html>
_______________________________________________
Player-dev mailing list
Player-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/player-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic