[patches] constification 1/2

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

 



On Mon, 2018-05-28 at 01:49 +0100, jnqnfe at gmail.com wrote:
> Constification patch set ** 1 of 2 **
> 
> Collection of 16 patches constifying pointers in various parts of the
> API.
> 
> This collection of patches has interdependencies, they must be applied
> in (roughly) the given order.
> 
> These start off with constifying some core hashmap functions, which
> then allows various proplist related functions to be changed. A couple
> of tagstruct functions are in there, and finally a couple of
> context+proplist related functions.
> 
> I have not been in a position to try and compile these changes. I have
> identified one possible problem - the hashmap.c BY_HASH macro - I'm not
> certain offhand if a const version will be required or if the compiler
> will be happy casting as is. Otherwise I'm fairly certain there are no
> (obvious) issues.

Thanks! I pushed these to the "next" branch. The only issue was in the
last patch - the function reused the constified variable when creating
a new proplist, and the compiler didn't like when that temporary
proplist was freed. I took the liberty of amending your patch with a
fix.

-- 
Tanu

https://liberapay.com/tanuk
https://www.patreon.com/tanuk


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

  Powered by Linux