[PATCH] Please review RAOP2 support :)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 18 Sep 2014 19:17:53 +0600, "Alexander E. Patrakov"
<patrakov at gmail.com> wrote:

> I had a look at the diff between the master and the end result. Too
> much whitespace changes and other non-functional changes (e.g. I
> don't understand why the definition of struct userdata has to be
> moved up in odule-raop-discover.c). I have not looked at the
> individual commits.

Most of the whitespace changes are actually extra whitespace removal. Do
you think I should revert those? I agree that they make the diff
longer, but that seems like wasting the time of the people who did
these fixes...

> Also, you have some "unimplemented" stuff like AAC encoding. Could
> you please remove it? We can always use ALAC, and already have a 
> presumably-working standalone "encoder" (that just slaps a header
> with an is-not-compressed bit) implementation in write_ALAC_data().

Sure.

-- 
Colin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20140918/80be4b9d/attachment.sig>


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux