summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefano Pigozzi <stefano.pigozzi@gmail.com>2013-07-01 08:43:22 +0200
committerStefano Pigozzi <stefano.pigozzi@gmail.com>2013-07-22 21:53:18 +0200
commit5a4ae428922076b8b741a2d56c3ba5a6865bc3fb (patch)
treea29c7441fa485a5f2b44fd85d9db1f5fea6c6ceb
parentd3fb585b58710a478c218ccfda5baa54b0e33a67 (diff)
downloadmpv-5a4ae428922076b8b741a2d56c3ba5a6865bc3fb.tar.bz2
mpv-5a4ae428922076b8b741a2d56c3ba5a6865bc3fb.tar.xz
ao_coreaudio: change all ++var to var++
Luckily they all were inside for loops so the functionality does not actually change.
-rw-r--r--audio/out/ao_coreaudio.c8
-rw-r--r--audio/out/ao_coreaudio_common.c12
2 files changed, 8 insertions, 12 deletions
diff --git a/audio/out/ao_coreaudio.c b/audio/out/ao_coreaudio.c
index e88f270afa..9768a5d4da 100644
--- a/audio/out/ao_coreaudio.c
+++ b/audio/out/ao_coreaudio.c
@@ -217,7 +217,7 @@ static void print_help(void)
int devs_n = devs_size / sizeof(AudioDeviceID);
- for (int i = 0; i < devs_n; ++i) {
+ for (int i = 0; i < devs_n; i++) {
char *name;
OSStatus err =
GetAudioPropertyString(devs[i], kAudioObjectPropertyName, &name);
@@ -461,8 +461,7 @@ static int init_digital(struct ao *ao, AudioStreamBasicDescription asbd)
int streams_n = size / sizeof(AudioStreamID);
- // TODO: ++i is quite fishy in here. Investigate!
- for (int i = 0; i < streams_n && d->stream_idx < 0; ++i) {
+ for (int i = 0; i < streams_n && d->stream_idx < 0; i++) {
bool digital = AudioStreamSupportsDigital(streams[i]);
if (digital) {
@@ -485,8 +484,7 @@ static int init_digital(struct ao *ao, AudioStreamBasicDescription asbd)
d->stream = streams[i];
d->stream_idx = i;
- // TODO: ++j is fishy. was like this in the original code. Investigate!
- for (int j = 0; j < formats_n; ++j)
+ for (int j = 0; j < formats_n; j++)
if (AudioFormatIsDigital(asbd)) {
// select the digital format that has exactly the same
// samplerate. If an exact match cannot be found, select
diff --git a/audio/out/ao_coreaudio_common.c b/audio/out/ao_coreaudio_common.c
index 6f35733c63..a3b7a741e3 100644
--- a/audio/out/ao_coreaudio_common.c
+++ b/audio/out/ao_coreaudio_common.c
@@ -235,7 +235,7 @@ static int AudioStreamSupportsDigital(AudioStreamID stream)
}
const int n_formats = size / sizeof(AudioStreamRangedDescription);
- for (int i = 0; i < n_formats; ++i) {
+ for (int i = 0; i < n_formats; i++) {
AudioStreamBasicDescription asbd = formats[i].mFormat;
ca_print_asbd("supported format:", &(asbd));
if (AudioFormatIsDigital(asbd)) {
@@ -264,7 +264,7 @@ static int AudioDeviceSupportsDigital(AudioDeviceID device)
}
const int n_streams = size / sizeof(AudioStreamID);
- for (int i = 0; i < n_streams; ++i) {
+ for (int i = 0; i < n_streams; i++) {
if (AudioStreamSupportsDigital(streams[i])) {
free(streams);
return CONTROL_OK;
@@ -280,11 +280,9 @@ static OSStatus ca_property_listener(AudioObjectPropertySelector selector,
const AudioObjectPropertyAddress addresses[],
void *data)
{
- // TODO: ++i seems wrong in this context. Check out if it was a programmer
- // mistake
void *talloc_ctx = talloc_new(NULL);
- for (int i = 0; i < n_addresses; ++i) {
+ for (int i = 0; i < n_addresses; i++) {
if (addresses[i].mSelector == selector) {
ca_msg(MSGL_WARN, "event: property %s changed\n",
fourcc_repr(talloc_ctx, selector));
@@ -418,8 +416,8 @@ static int AudioStreamChangeFormat(AudioStreamID i_stream_id,
* it is also not Atomic, in its behaviour.
* Therefore we check 5 times before we really give up. */
bool format_set = CONTROL_FALSE;
- for (int i = 0; !format_set && i < 5; ++i) {
- for (int j = 0; !stream_format_changed && j < 50; ++j)
+ for (int i = 0; !format_set && i < 5; i++) {
+ for (int j = 0; !stream_format_changed && j < 50; j++)
mp_sleep_us(10000);
if (stream_format_changed) {