--- a/libavcodec/mlpdsp.c
+++ b/libavcodec/mlpdsp.c
@@ -89,10 +89,46 @@ void ff_mlp_rematrix_channel(int32_t *samples,
+
+int32_t ff_mlp_pack_output(int32_t lossless_check_data,
This function is not used outside of the file, so it can be made
static and the ff_ prefix can be removed.

It's used from several places in arm/mlpdsp_init_arm.c.

+    int32_t *data_32 = (int32_t *)data;
+    int16_t *data_16 = (int16_t *)data;
pointless void* casts

Fair enough, those were cut-and-pastes from their original location.

+            lossless_check_data ^= (sample & 0xffffff) << mat_ch;
+            if (is32) *data_32++ = sample << 8;
+            else      *data_16++ = sample >> 8;
Please break the lines.

OK, can do.

Ben
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to