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@lists.xen.org http://lists.xen.org/xen-devel