Commit af58de65 authored by Max Kellermann's avatar Max Kellermann

simplified code in the ogg decoder plugin

Return early when the player thread sent us a command. This saves one level of indentation.
parent 940ecf53
...@@ -232,32 +232,32 @@ static int oggvorbis_decode(struct decoder * decoder, InputStream * inStream) ...@@ -232,32 +232,32 @@ static int oggvorbis_decode(struct decoder * decoder, InputStream * inStream)
callbacks.close_func = ogg_close_cb; callbacks.close_func = ogg_close_cb;
callbacks.tell_func = ogg_tell_cb; callbacks.tell_func = ogg_tell_cb;
if ((ret = ov_open_callbacks(&data, &vf, NULL, 0, callbacks)) < 0) { if ((ret = ov_open_callbacks(&data, &vf, NULL, 0, callbacks)) < 0) {
if (decoder_get_command(decoder) != DECODE_COMMAND_STOP) { if (decoder_get_command(decoder) != DECODE_COMMAND_NONE)
switch (ret) { return 0;
case OV_EREAD:
errorStr = "read error"; switch (ret) {
break; case OV_EREAD:
case OV_ENOTVORBIS: errorStr = "read error";
errorStr = "not vorbis stream"; break;
break; case OV_ENOTVORBIS:
case OV_EVERSION: errorStr = "not vorbis stream";
errorStr = "vorbis version mismatch"; break;
break; case OV_EVERSION:
case OV_EBADHEADER: errorStr = "vorbis version mismatch";
errorStr = "invalid vorbis header"; break;
break; case OV_EBADHEADER:
case OV_EFAULT: errorStr = "invalid vorbis header";
errorStr = "internal logic error"; break;
break; case OV_EFAULT:
default: errorStr = "internal logic error";
errorStr = "unknown error"; break;
break; default:
} errorStr = "unknown error";
ERROR("Error decoding Ogg Vorbis stream: %s\n", break;
errorStr);
return -1;
} }
return 0; ERROR("Error decoding Ogg Vorbis stream: %s\n",
errorStr);
return -1;
} }
audio_format.bits = 16; audio_format.bits = 16;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment