07.04.2014 18:24, Tanu Kaskinen wrote:
+/* Called from the main thread. */
+static void set_reference_volume_internal(pa_sink *s, const pa_cvolume
*volume) {
+ pa_cvolume old_volume;
+ char old_volume_str[PA_CVOLUME_SNPRINT_VERBOSE_MAX];
+ char new_volume_str[PA_CVOLUME_SNPRINT_VERBOSE_MAX];
+
+ pa_assert(s);
+ pa_assert(volume);
Note the assertions.
+
+/* Called from the main thread. */
+void pa_sink_set_reference_volume_direct(pa_sink *s, const pa_cvolume *volume)
{
+ pa_assert(s);
+ pa_assert(volume);
+
+ set_reference_volume_internal(s, volume);
+}
What's the point of having this function in this patch? As demonstrated
above, set_reference_volume_internal does the same assertions.
Same for sources.
The patch does look like an equivalent refactoring.
--
Alexander E. Patrakov
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss