Re: [Xen-devel] [PATCH 2/7] libxc: Use const pointer in local_file_dump()

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Jennifer Herbert writes ("[Xen-devel] [PATCH 2/7] libxc: Use const pointer in 
local_file_dump()"):
> By adding the const keyword, it is clearer to people and static analysis
> tools that no changes to the data are to be made.

Would it be wrong for a future patch to add a field to dump_args which
gets modified ?  AFAICT the answer is `no'.  So I don't understand why
it ought to be const.

Does your Coverity instance complain about every instance where a
struct exists which is not marked const but which could be ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel