On 22/07/16 11:14, Arun Raghavan wrote:
On Wed, 8 Jun 2016, at 03:49 PM, Pierre Ossman wrote:
Hopefully that's sufficient for now?
Yes, it is. I think the patches you posted look good. Just one question
-- you moved missing and requested to being size_ts. Is there a reason
you don't leave them as int64_ts and avoid the casts in comparison?
This was to properly solidify the concept that these can never be
negative, which was the source of the earlier confusion. size_t seemed
to be the type used in the rest of the API for such things.
Thanks for merging. Any chance you could have a look at my tunnel module
patches next? :)
Regards
--
Pierre Ossman Software Development
Cendio AB http://cendio.com
Teknikringen 8 http://twitter.com/ThinLinc
583 30 Linköping http://facebook.com/ThinLinc
Phone: +46-13-214600 http://plus.google.com/+CendioThinLinc
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
_______________________________________________
pulseaudio-discuss mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
