[pulseaudio-discuss] [PATCH] Please review RAOP2 support :)

Colin Leroy colin at colino.net
Thu Sep 18 06:44:08 PDT 2014


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

> Well, if you prefer to have them, then OK, let's have them (and only 
> them) at the beginning and only functional changes at the end. If
> some patch adds badly-formatted code only to be fixed up later,
> please squash the formatting fixup into the original patch.
> 
> >  
> >> 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.  
> 
> Also forgot to say: some patches in the series look like fixups for 
> others. E.g. "raop: Add a MD5 hashing fuction" and "raop: Correct the 
> MD5 hashing helper function". I think it is a good idea to squash
> such fixups.

OK, I see what you mean. That means I probably have to start from
pulseaudio/master, apply existing patches manually and commit them in a
new form, without referencing the original raop2 branch. Is this
OK ?

Thanks,
-- 
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/1734e3a7/attachment.sig>


More information about the pulseaudio-discuss mailing list