claunia/libcdio-osx
Archived
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Correct wording in cdda-player comments. For Christopher Yeleighton,

Browse Source 
among others.
This commit is contained in:
R. Bernstein
2011-05-18 04:46:36 -04:00
parent ed6311426d
commit 46876a2040
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/cdda-player.c
View File

@@ -372,7 +372,8 @@ set_volume_level(CdIo_t *p_cdio, uint8_t i_level)
} }
/* Add 1 to volume level. If we are at the min value, nothing is done. /* Subtract one from the volume level. If we are at the minimum value,
* nothing is done.
We used to wrap at the boundaries but this is probably wrong because We used to wrap at the boundaries but this is probably wrong because
is assumes someone: is assumes someone:
@@ -381,9 +382,10 @@ set_volume_level(CdIo_t *p_cdio, uint8_t i_level)
See issue #33333 See issue #33333
f volume level is undefined, then this means we could not get the If the volume level is undefined, then this means we could not get
current value and we'll' set it to 50 the midway point. Return the current value and we'll' set it to 50 the midway point.
status of setting the volume level. */
Return the status of setting the volume level. */
static bool static bool
decrease_volume_level(CdIo_t *p_cdio) decrease_volume_level(CdIo_t *p_cdio)
{ {
@@ -392,8 +394,8 @@ decrease_volume_level(CdIo_t *p_cdio)
return set_volume_level(p_cdio, --i_volume_level); return set_volume_level(p_cdio, --i_volume_level);
} }
/* Subtract 1 to volume level. If we are at the max or min value, /* Add 1 to the volume level. If we are at the maximum value, nothing
nothing is done. is done.
We used to wrap at the boundaries but this is probably wrong because We used to wrap at the boundaries but this is probably wrong because
is assumes someone: is assumes someone:
@@ -405,13 +407,13 @@ decrease_volume_level(CdIo_t *p_cdio)
If volume level is undefined, then this means we could not get the If volume level is undefined, then this means we could not get the
current value and we'll' set it to 50 the midway point. current value and we'll' set it to 50 the midway point.
Return status of setting the volume level. */ Return the status of setting the volume level. */
static bool static bool
increase_volume_level(CdIo_t *p_cdio) increase_volume_level(CdIo_t *p_cdio)
{ {
if (i_volume_level == -1) i_volume_level = 49; if (i_volume_level == -1) i_volume_level = 49;
if (i_volume_level <= 0) i_volume_level = 0; if (i_volume_level <= 0) i_volume_level = 0;
if (i_volume_level >= 98) i_volume_level = 98; if (i_volume_level > 98) i_volume_level = 98;
return set_volume_level(p_cdio, ++i_volume_level); return set_volume_level(p_cdio, ++i_volume_level);
} }