Bug 4095 – Volume sometimes found set to n.5 after upgrade

Bug 4095 - Volume sometimes found set to n.5 after upgrade

Summary:

Volume sometimes found set to n.5 after upgrade

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Player UI
Version:	6.5b2
Platform:	PC Windows XP

Importance:	P2 trivial (vote)
Target Milestone:	---
Assigned To:	Chris Owens

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-09-12 14:24 UTC by Chris Owens
Modified:	2009-09-08 09:31 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
simple int mask for display (350 bytes, patch) 2006-09-12 23:57 UTC, KDF	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Chris Owens 2006-09-12 14:24:22 UTC

John Coyne reported this one.  After upgrading from 6.3.1 to 6.5, he noticed his volume was set to something.5.

After lowering his volume to zero, the problem went away.

Comment 1 KDF 2006-09-12 15:16:16 UTC

probably a result of the older 40-step setting.  100/40 = 2.5 so the volume was set at increments of 2.5.  
if it's important, we could probably add a simple int() when loading the pref, or via an upgrade script.

Comment 2 Chris Owens 2006-09-12 15:41:47 UTC

I'll cc Dean since he was the one that asked John to file it.

I judge from his interest that if it really is as simple as adding an int() when loading prefs that we should go ahead and do that.

Comment 3 Blackketter Dean 2006-09-12 16:14:18 UTC

Subject: Re:  Volume sometimes found set to n.5 after upgrade

The int() should probably be done going to the display, rather than  
touching the settings, but yes, this is a relatively simple fix.

Comment 4 KDF 2006-09-12 23:57:51 UTC

Created attachment 1520 [details]
simple int mask for display

shows on display as an int.

Comment 5 Blackketter Dean 2006-09-13 09:39:11 UTC

Subject: Re:  Volume sometimes found set to n.5 after upgrade

Go for it!

Comment 6 KDF 2006-09-13 23:19:15 UTC

fixed at change 9702 in trunk and 6.5 at change 9703