[Vortex] [PATCH] fix vortex_channel_new prototype

Jens Alfke jens at mooseyard.com
Wed Dec 12 23:05:29 CET 2007


On 5 Dec '07, at 9:06 AM, Francis Brosnan Blazquez wrote:

>> vortex_channel_new doesn't need write access to profile parameter.
>> Attached patch fixes that.
>> Maybe it'll be rejected due to API breakage though.
>
> Thanks for the patch, it is now applied since the profile parameter
> usually receives a static string.

There are still many other parameters in the public API that should  
similarly be changed from "char*" to "const char*". A few dozen, I  
think.

Is anyone planning to fix these? If I fixed them and submitted a  
patch, would it be accepted? (Just like the above patch, this isn't an  
API change because existing client code, whether source or binary,  
will still work.)

--Jens



More information about the Vortex mailing list